Skip to content

Enable offline entitlements#987

Merged
tonidero merged 1 commit into
mainfrom
enable-offline-entitlements
May 23, 2023
Merged

Enable offline entitlements#987
tonidero merged 1 commit into
mainfrom
enable-offline-entitlements

Conversation

@tonidero

@tonidero tonidero commented Apr 27, 2023

Copy link
Copy Markdown
Contributor

Description

This removes the flag that was disabling offline entitlements, so we actually start using offline entitlements

TODO

@tonidero tonidero added HOLD pr:feat A new feature labels Apr 27, 2023
@tonidero tonidero force-pushed the enable-offline-entitlements branch from 250d84f to a1df7e4 Compare April 28, 2023 08:07
@tonidero tonidero removed the HOLD label Apr 28, 2023
@tonidero tonidero marked this pull request as ready for review April 28, 2023 08:10
@tonidero tonidero requested a review from a team April 28, 2023 08:11
@tonidero tonidero added the HOLD label Apr 28, 2023
@vegaro

vegaro commented Apr 28, 2023

Copy link
Copy Markdown
Member

Do we need #959 first? I am going to work on rebasing that PR today

@tonidero

tonidero commented May 3, 2023

Copy link
Copy Markdown
Contributor Author

Do we need #959 first? I am going to work on rebasing that PR today

You're right! Holding this PR until that is merged as well

@tonidero tonidero force-pushed the enable-offline-entitlements branch from a1df7e4 to c6f739b Compare May 16, 2023 09:25
@codecov

codecov Bot commented May 16, 2023

Copy link
Copy Markdown

Codecov Report

Merging #987 (8d526d4) into main (c924d62) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #987      +/-   ##
==========================================
+ Coverage   85.35%   85.38%   +0.03%     
==========================================
  Files         168      168              
  Lines        5997     5989       -8     
  Branches      837      834       -3     
==========================================
- Hits         5119     5114       -5     
+ Misses        546      544       -2     
+ Partials      332      331       -1     
Impacted Files Coverage Δ
.../java/com/revenuecat/purchases/common/AppConfig.kt 82.22% <ø> (-0.39%) ⬇️
...otlin/com/revenuecat/purchases/PurchasesFactory.kt 82.05% <ø> (-0.16%) ⬇️
...at/purchases/strings/OfflineEntitlementsStrings.kt 0.00% <ø> (ø)
.../offlineentitlements/OfflineEntitlementsManager.kt 93.02% <100.00%> (-0.86%) ⬇️

... and 1 file with indirect coverage changes

@tonidero tonidero force-pushed the enable-offline-entitlements branch from c6f739b to 8d526d4 Compare May 19, 2023 10:06
@tonidero tonidero removed the HOLD label May 23, 2023
@tonidero tonidero merged commit 594138c into main May 23, 2023
@tonidero tonidero deleted the enable-offline-entitlements branch May 23, 2023 08:46
tonidero added a commit that referenced this pull request May 23, 2023
### Description
Depends on #987 

This PR adds some integration tests checking some basic behaviors of
offline entitlements. Mostly making sure the mode is turn on and off as
expected.

<!-- Describe your changes in detail -->
<!-- Please describe in detail how you tested your changes -->
This was referenced May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:feat A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants