Skip to content

Unsubscribe from survey updates after clearing user session - #3888

Merged
andreia-ferreira merged 6 commits into
masterfrom
andreia/3867/unsubscribe-on-logout
Aug 20, 2026
Merged

Unsubscribe from survey updates after clearing user session#3888
andreia-ferreira merged 6 commits into
masterfrom
andreia/3867/unsubscribe-on-logout

Conversation

@andreia-ferreira

Copy link
Copy Markdown
Collaborator

Towards #3867

Similar to #3876, but this PR focuses on unsubscribing from firebase survey updates when a user logs out.

This also introduces a small refactor to move the ClearUserSessionUseCase to :core:domain. Its android specific dependencies needed to be removed, so a new method was introduced in the UserRepository to clear all user data, not only the shared preferences, but also the data stored in the Room database.

@shobhitagarwal1612 PTAL?

@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.79%. Comparing base (a831afb) to head (74aee2a).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3888      +/-   ##
============================================
+ Coverage     68.69%   68.79%   +0.10%     
- Complexity     1926     1929       +3     
============================================
  Files           412      412              
  Lines         10968    10970       +2     
  Branches       1432     1432              
============================================
+ Hits           7534     7547      +13     
+ Misses         2686     2675      -11     
  Partials        748      748              
Files with missing lines Coverage Δ
...roundplatform/android/repository/UserRepository.kt 83.05% <100.00%> (+1.91%) ⬆️
...rg/groundplatform/android/ui/main/MainViewModel.kt 65.57% <ø> (ø)
...rm/domain/usecases/user/ClearUserSessionUseCase.kt 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andreia-ferreira
andreia-ferreira merged commit 7dd6f7d into master Aug 20, 2026
19 checks passed
@andreia-ferreira
andreia-ferreira deleted the andreia/3867/unsubscribe-on-logout branch August 20, 2026 16:27
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