Skip to content

remove pro 25#1660

Merged
Kastier1 merged 1 commit intomainfrom
carlos/remove-25
Oct 17, 2025
Merged

remove pro 25#1660
Kastier1 merged 1 commit intomainfrom
carlos/remove-25

Conversation

@carlosabadia
Copy link
Collaborator

No description provided.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_TABLE in pcweb/constants.py:104
  • Updated default credits in ProTierState from "Pro 25" to "Pro 50" in pcweb/pages/pricing/plan_cards.py:27
  • Updated fallback tier in selected_tier computed property from "Pro 25" to "Pro 50" in pcweb/pages/pricing/plan_cards.py:45
  • Changes are consistent and all references to "Pro 25" have been removed
  • Other files using PRO_TIERS_TABLE (like slider_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
Loading

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@Kastier1 Kastier1 merged commit d340a9a into main Oct 17, 2025
10 checks passed
@Kastier1 Kastier1 deleted the carlos/remove-25 branch October 17, 2025 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments