Merged
Conversation
Contributor
There was a problem hiding this comment.
Greptile Overview
Summary
Removes the "Pro 25" pricing tier from the application, adjusting the default tier selection to "Pro 50".
- Removed "Pro 25" entry from
PRO_TIERS_TABLEinpcweb/constants.py:104 - Updated default credits in
ProTierStatefrom "Pro 25" to "Pro 50" inpcweb/pages/pricing/plan_cards.py:27 - Updated fallback tier in
selected_tiercomputed property from "Pro 25" to "Pro 50" inpcweb/pages/pricing/plan_cards.py:45 - Changes are consistent and all references to "Pro 25" have been removed
- Other files using
PRO_TIERS_TABLE(likeslider_calculator.py) dynamically iterate over tiers and are not affected
Confidence Score: 5/5
- This PR is safe to merge with minimal risk
- The changes are straightforward and surgical - removing a single pricing tier and updating two related default/fallback references. All references to "Pro 25" have been properly removed from the codebase. The changes maintain consistency across the pricing logic.
- No files require special attention
Important Files Changed
File Analysis
| Filename | Score | Overview |
|---|---|---|
| pcweb/constants.py | 5/5 | Removed "Pro 25" tier from PRO_TIERS_TABLE dictionary |
| pcweb/pages/pricing/plan_cards.py | 5/5 | Updated default tier and fallback from "Pro 25" to "Pro 50" |
Sequence Diagram
sequenceDiagram
participant User
participant ProTierState
participant PRO_TIERS_TABLE
participant BillingPage
Note over PRO_TIERS_TABLE: Pro 25 tier removed
Note over PRO_TIERS_TABLE: Now starts at Pro 50
User->>ProTierState: Initialize state
ProTierState->>PRO_TIERS_TABLE: Get default credits for "Pro 50"
PRO_TIERS_TABLE-->>ProTierState: Return 1000 credits
Note over ProTierState: Default credits = 1000 (was 500)
User->>ProTierState: Update credits via slider
ProTierState->>ProTierState: Call update_credits()
ProTierState->>ProTierState: Calculate selected_tier
ProTierState->>PRO_TIERS_TABLE: Find tier matching credits
alt Matching tier found
PRO_TIERS_TABLE-->>ProTierState: Return tier data
else No match found
PRO_TIERS_TABLE-->>ProTierState: Fallback to "Pro 50"
Note over ProTierState: Fallback changed from Pro 25 to Pro 50
end
User->>ProTierState: Click "Start with Pro plan"
ProTierState->>BillingPage: redirect_to_billing(yearly)
BillingPage-->>User: Navigate to billing with selected tier
2 files reviewed, no comments
Kastier1
approved these changes
Oct 17, 2025
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.
No description provided.