fix(staging): pin website liveness probe to port 4000 - #1111
Closed
think-back wants to merge 3 commits into
Closed
think-back wants to merge 3 commits into
think-back wants to merge 3 commits into
Conversation
Publish the new 13/45/170 monthly value contract and remove short-window marketing. Keep actual checkout discounts sourced from backend quotes. Constraint: Future purchases use new plan and Stripe Price versions while historical entitlements and renewals retain their snapshots Rejected: Updating existing Go/Pro/Max plan rows | would change grandfathered users and recurring bindings Confidence: high Scope-risk: moderate Directive: Keep historical plan rows and 5h/7d compatibility paths immutable Tested: Wallet 83 tests; website 48 tests and typecheck; both production builds; local website and console previews; staging entitlement snapshot audit Not-tested: End-to-end Stripe checkout will run after the staging deployment
|
Constraint: The staging website listens on port 4000 while its inherited Cloud Run liveness probe had drifted to 8080. Rejected: Rely on Terraform to repair the probe | The service lifecycle intentionally ignores probe changes. Confidence: high Scope-risk: narrow Directive: Keep the staging deploy probe aligned with the website container port. Tested: git diff --check; live Cloud Run revision probes / on port 4000 successfully Not-tested: Full workflow run occurs after merge to staging
Constraint: The user explicitly excluded frontend changes from the new subscription entitlement rollout. Rejected: Drop the branch entirely | The independent staging liveness repair must remain available for main. Confidence: high Scope-risk: narrow Directive: Keep plan entitlement configuration separate from public pricing and wallet presentation unless explicitly requested. Tested: all 14 frontend files exactly match the pre-change parent; git diff --check; staging liveness flag remains on port 4000 Not-tested: No production deployment is authorized
|
Collaborator
Author
|
Superseded by #1116, recreated cleanly from the latest main branch with only the staging liveness workflow change. All subscription frontend changes have been removed as requested. |
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
Scope
This PR remains draft; the equivalent fix is already merged and verified on the staging branch.