Conversation
- Add "type": "module" to package.json to enable ES module support - Extract axios, i18n, and Sentry configuration into src/services/api.ts service module - Lazy-load apex charts library
- Add PaymentPlanSwitchOperation model to track Stripe subscription plan changes durably - Implement plan switching logic - Enables idempotent payment plan switches with Stripe sync
|
Created image with name |
rechner
commented
Sep 19, 2026
| xsrfCookieName: 'csrftoken', | ||
| xsrfHeaderName: 'X-CSRFTOKEN', | ||
| }); | ||
| import { api } from '../services/api'; |
Member
Author
There was a problem hiding this comment.
These frontend changes are unrelated to fix compiler warnings
rechner
commented
Sep 19, 2026
| 'axios', | ||
| 'routeGuards', | ||
| 'capacitor', | ||
| 'apexcharts', |
Member
Author
There was a problem hiding this comment.
apexcharts is hulking, reworked to lazy load on the necessary page instead.
- Add tests for stripe helper functions accepting dict and string objects - Add test for idempotentcy when profile already at target plan - Add test for profile drift detection - Test for database failure tolerance in error recording - Add test for unconfirmed stripe update retry logic - Add tests for plan switch validation (missing plan ID, locked member, same plan) - Add tests for pending and failed operation rejection - Add tests for subscription item count and price validation - Add tests for terminal recovery failure reporting - Improve test coverage for payment plan switch recovery and error handling
|
Created image with name |
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.
Closes #41, focused on switching to another payment plan in the same tier with the same subscription period. Mostly to enable Sudoroom members existing pay-what-you-can dues model.
Complicated to include a durable recovery path. Should hopefully tell you when something is wrong/unrecoverable.
PaymentPlanSwitchOperationmodel to track Stripe subscription plan changes durably