An upgrade confirmed across a renewal answers "Try again in a moment" instead of showing fresh figures #292
Open
opened 2026-10-30 00:30:57 +00:00 by cgalo5758
·
0 comments
Labels
Clear labels
accessibility
area/billing
area/catalog
area/discourse
area/domains
area/entitlements
area/fedwiki
area/identity
area/integrations
area/licensing
area/member-ui
area/meta
area/operator-ui
area/ops
area/testing
duplicate
good-first-issue
invalid
privacy
security
upstream
wontfix
A barrier for people using assistive technology or a keyboard alone.
The Stripe mirror, checkout, subscriptions, invoices, fulfillment.
Products, prices, plan ladders, purchasability.
The Discourse integration.
The domains registry, claims, placements, the certificate ask.
Entitlement sets, rules, grants, pools, provisioning.
The Federated Wiki integration and farm sync.
Sign-in, sessions, persons, organizations, workspaces, roles.
The provider registry, outbox and webhooks in general.
Licenses, the contributor agreement, SPDX headers.
Member pages.
The repository itself, its contributing guide, CI, the tracker and the workflow.
Operator pages, forms, lists, the design system.
Deployment, configuration, migrations, workflows, instance settings.
The test stack, screens, lint, walkthroughs.
Closed because another issue already covers it.
Small, self-contained, and explained enough to be a first contribution.
Closed because it is not a ticket for this repository.
Touches what a person's data reveals.
Touches authentication, authorization, secrets or data exposure.
Waits on another repository or project before it can move.
Closed because it will not be done, with the reason in the last comment.
kind
bug
The software does something other than what it promises; closed when it again does what it promises.
kind
debt
Code, tests or tooling to clean up with nothing visible changing; closed when they are cleaner.
kind
design
A question to settle before work can be defined; closed when the decision is written down.
kind
docs
Documentation that is wrong or missing; closed when it says the right thing.
kind
enhancement
Something the software does not do yet; closed when it does.
priority
critical
Blocks the active milestone or harms members now.
priority
high
Next in line inside the active milestone.
priority
low
Inside the active milestone, when nothing else is left.
priority
medium
Inside the active milestone, after the high ones.
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: wiki-cafe/member-console#292
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
What happens
A member opens an upgrade's confirmation, which shows a prorated charge computed for a moment in the current billing period, and confirms it after the subscription has renewed. The upgrade sends that moment to Stripe as the proration date, and Stripe refuses a proration date before the current period's start with a 400. The member reads the generic "We couldn't complete that change. Try again in a moment." The upgrade checks only how old the preview is before it calls Stripe, not whether the period it was computed in has ended.
The switch of a paused plan already makes this check after it reads the subscription, and answers a stale preview by showing fresh figures, the same answer a preview that is too old gets.
What should happen
An upgrade whose preview was computed before the subscription's current period started is answered as a stale preview: the member sees the confirmation again with the new period's figures, and nothing is sent to Stripe.
Where
upgradeNow,internal/fulfillment/plan_change.go(the check againstPreviewValidFor); the paused form's check ininternal/fulfillment/paused_switch.goWhy it matters
A member who confirms an upgrade around their renewal is told to retry something that fails the same way until they reopen the page, with no sign that the figures changed.