CC-628: Refresh Customer Center UI after subscription cancellation#3061
Merged
facumenzella merged 6 commits intoFeb 6, 2026
Conversation
- Add lifecycle-aware refresh when activity resumes after being paused - Add isRefreshing state to show loading indicator during refresh - Add refreshCustomerCenter() method that keeps content visible while refreshing - Match iOS behavior: refresh when returning from manage subscriptions screen - Show subtle loading indicator and dim content during refresh (similar to iOS)
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
skydoves
reviewed
Feb 3, 2026
skydoves
left a comment
Member
There was a problem hiding this comment.
All looks good 🙌 I just left some suggestions, but it's not very critical :)
- Use LifecycleResumeEffect instead of DisposableEffect with LifecycleEventObserver - Add isRefreshing check to prevent race condition - Remove unused size import - Remove unnecessary padding from CircularProgressIndicator Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3061 +/- ##
=======================================
Coverage 79.18% 79.18%
=======================================
Files 342 342
Lines 13680 13680
Branches 1841 1841
=======================================
Hits 10833 10833
Misses 2088 2088
Partials 759 759 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
vegaro
requested changes
Feb 3, 2026
- Move refresh logic from Composable to ViewModel to survive config changes - Use ON_STOP/ON_START with isChangingConfigurations check to distinguish real backgrounding from rotation - Add error logging when refresh fails Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
vegaro
reviewed
Feb 3, 2026
- Use non-deprecated LocalLifecycleOwner import from lifecycle-compose - Use context.getActivity() to safely handle ContextWrapper - Add empty line after refreshCustomerCenter() for consistency Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Member
Author
|
Addressed all feedback in 3b088dd:
|
vegaro
approved these changes
Feb 6, 2026
github-merge-queue Bot
pushed a commit
that referenced
this pull request
Feb 6, 2026
**This is an automatic release.** ## RevenueCatUI SDK ### Customer Center #### 🐞 Bugfixes * CC-628: Refresh Customer Center UI after subscription cancellation (#3061) via Facundo Menzella (@facumenzella) ### Paywallv2 #### 🐞 Bugfixes * Improve carousel performance with videos (#3070) via Cesar de la Vega (@vegaro) ### 🔄 Other Changes * Make networkName nullable in ad event data types (#3076) via Pol Miro (@polmiro) * Remove networkName from AdFailedToLoad event (#3074) via Pol Miro (@polmiro) Co-authored-by: revenuecat-ops <ops@revenuecat.com>
github-merge-queue Bot
pushed a commit
that referenced
this pull request
Mar 16, 2026
…ions (#3152) ## Summary - fix Customer Center duplicate reload on foreground by removing collector re-subscription auto-load and running initial load once in `init` - refresh Customer Center after returning from manage subscriptions and force a stronger refresh via `awaitSyncPurchases()` - keep existing lifecycle refresh behavior and add a one-time delayed follow-up refresh guard for post-Play propagation - remove temporary debug instrumentation logs added during investigation - add regression tests to verify refresh path uses sync ## Testing - ./gradlew :ui:revenuecatui:testDefaultsBc8DebugUnitTest --tests "*CustomerCenterViewModelTests" Follow-up of #3061 Closes CC-628 <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Medium Risk** > Touches Customer Center lifecycle/refresh behavior; mis-ordering lifecycle events or refresh gating could cause missed or duplicate reloads when foregrounding or returning from external subscription management. > > **Overview** > Improves Customer Center refresh behavior to better reflect subscription changes after users leave the app to manage/cancel subscriptions. > > Adds an `ON_RESUME` lifecycle hook and a one-shot `shouldRefreshOnResume` flag set when launching external subscription management, so returning triggers a targeted refresh while avoiding duplicate refreshes on `ON_START`. Refresh now preserves key UI/navigation state when updating data, and initial loading via the state flow is guarded to prevent repeated `loadCustomerCenter()` calls on re-subscription. > > Updates tests to cover the new resume-driven refresh path and assert refresh uses `awaitCustomerInfo(CacheFetchPolicy.FETCH_CURRENT)` (not `awaitSyncPurchases()`), plus a small newline fix in `MockPurchasesType`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 0bcac84. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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
3 participants
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
Fixes issue where Customer Center UI doesn't update after subscription cancellation.
Changes
isRefreshingstate toCustomerCenterState.Successto show loading indicator during refreshrefreshCustomerCenter()method that keeps content visible while refreshing (shows subtle loading indicator instead of full loading screen)Testing
Closes CC-628