Replace repository plan API with subscription plan endpoint#82
Merged
Conversation
- Deleted the `change_repository_plan.py` file, which included the implementation for upgrading or downgrading shared repository subscription plans. - Updated the `get_graph_subscription.py` file to reflect changes in the subscription retrieval documentation, clarifying that it now handles both graph and shared repository subscriptions. - Enhanced the `GraphSubscriptionResponse` model by adding an `operation_id` field for tracking asynchronous tier changes. These changes streamline the API by removing unused functionality and improving the clarity of existing endpoints.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Refactors the subscription management API by replacing the repository-scoped
change_repository_planendpoint with a broaderchange_subscription_planendpoint, and enhances the graph subscription response model with additional fields.Key Accomplishments
change_repository_plan.pytochange_subscription_plan.py, shifting from a repository-centric plan change to a subscription-level plan change. The new implementation includes expanded logic and improved handling (~80 lines added, ~36 removed).GraphSubscriptionResponseto capture richer subscription metadata, improving the data available to consumers of the subscription API.get_graph_subscriptionendpoint: Modified the graph subscription retrieval logic to align with the new response model structure and ensure consistency across the subscription API surface.Breaking Changes
change_repository_planAPI endpoint has been removed and replaced withchange_subscription_plan. Any clients or integrations referencing the old endpoint or its associated models will need to be updated.GraphSubscriptionResponsemodel has new fields, which may affect deserialization or schema validation in downstream consumers depending on how they handle unknown/additional fields.Testing Notes
change_subscription_planendpoint correctly handles plan change requests that were previously routed throughchange_repository_plan.get_graph_subscriptionendpoint returns the updated response model with all new fields populated correctly.change_repository_planendpoint remain in client code or integration layers.Infrastructure Considerations
🤖 Generated with Claude Code
Branch Info:
feature/graph-subscription-changemainCo-Authored-By: Claude noreply@anthropic.com