Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

15. Subscriptions: Properly sign out users and cache state #2520

Merged
merged 2 commits into from
Mar 1, 2024

Conversation

afterxleep
Copy link
Collaborator

@afterxleep afterxleep commented Feb 29, 2024

Task/Issue URL: https://app.asana.com/0/0/1206707680638927/f

Description:

  • If the subscription has become inactive, the user may still see the options in settings. This forces a signout in that case and caches the subscription state data properly.

Steps to test this PR:

  1. Purchase a sub
  2. Close setings
  3. Wait for it to expire
  4. Open the app
  5. Subscription options should not be shown. (Only purchase options)

Internal references:

Software Engineering Expectations
Technical Design Template

@github-actions github-actions bot added the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Feb 29, 2024
@afterxleep afterxleep requested a review from miasma13 February 29, 2024 19:37
@miasma13
Copy link
Contributor

miasma13 commented Mar 1, 2024

For the described scenario it worked as expected. Though for the first time I left the app when subscription was active with the settings open. And returning to the app the settings didn't refresh.
Not sure how hard it would be but WDYT about refreshing the subscription part of settings when returning to the app.

@afterxleep afterxleep removed the bot: not in app board Added by automation for pull requests with tasks not added to iOS App Board Asana project label Mar 1, 2024
@afterxleep
Copy link
Collaborator Author

For the described scenario it worked as expected. Though for the first time I left the app when subscription was active with the settings open. And returning to the app the settings didn't refresh. Not sure how hard it would be but WDYT about refreshing the subscription part of settings when returning to the app.

Yeap, adding notification subscribers in the next PR

@afterxleep afterxleep merged commit 3fbcab6 into main Mar 1, 2024
13 checks passed
@afterxleep afterxleep deleted the daniel/subscriptions/15.logout-on-inactive branch March 1, 2024 14:39
quanganhdo pushed a commit that referenced this pull request Mar 4, 2024
Task/Issue URL: https://app.asana.com/0/0/1206707680638927/f

Description:
If the subscription has become inactive, the user may still see the options in settings. This forces a signout in that case and caches the subscription state data properly.
quanganhdo pushed a commit that referenced this pull request Mar 4, 2024
Task/Issue URL: https://app.asana.com/0/0/1206707680638927/f

Description:
If the subscription has become inactive, the user may still see the options in settings. This forces a signout in that case and caches the subscription state data properly.
samsymons added a commit that referenced this pull request Mar 6, 2024
# By Daniel Bernal (3) and others
# Via GitHub
* main:
  fix opening tabs with transitional (#2542)
  Integrate confirm entitlements endpoint (#2541)
  Makes dbSaveBloomFilterError daily and count (#2526)
  16. Subscription: Display "Activation in progress" message (#2535)
  Autofill support for deleting all passwords  (#2497)
  Bump BrowserServicesKit (#2532)
  Release 7.111.0-0 (#2534)
  if dax dialogs are showing then dismiss (#2506)
  Remove local BSK reference (#2529)
  15. Subscription sign-out notification observers and Design Feedback (#2528)
  Upgrade to Xcode 15.2 (#2517)
  Report Apple Ad attribution using pixel (#2510)
  15. Subscriptions: Properly sign out users and cache state (#2520)
  Check entitlement periodically and while rekeying NetP (#2461)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
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