Skip to content

Grant grace period for entitlements when the backend is offline#851

Merged
tonidero merged 3 commits into
entitlements-verificationfrom
toniricodiez/sdk-2895-create-a-grace-period-for-entitlements
Mar 8, 2023
Merged

Grant grace period for entitlements when the backend is offline#851
tonidero merged 3 commits into
entitlements-verificationfrom
toniricodiez/sdk-2895-create-a-grace-period-for-entitlements

Conversation

@tonidero

@tonidero tonidero commented Mar 8, 2023

Copy link
Copy Markdown
Contributor

Description

Completes SDK-2895.

Android equivalent of: RevenueCat/purchases-ios#2288

In this PR we are adding a 3 day grace period to entitlements expiration dates.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought about adding these extensions to production code, but since I wasn't planning to use them yet, I left them as test code for now. We can move them in the future if needed.

@tonidero tonidero marked this pull request as ready for review March 8, 2023 11:04
@tonidero tonidero requested a review from a team March 8, 2023 11:04
@codecov

codecov Bot commented Mar 8, 2023

Copy link
Copy Markdown

Codecov Report

Merging #851 (16f47b9) into entitlements-verification (86d1287) will increase coverage by 0.01%.
The diff coverage is 90.90%.

❗ Current head 16f47b9 differs from pull request most recent head a962240. Consider uploading reports for the commit a962240 to get more accurate results

@@                      Coverage Diff                      @@
##           entitlements-verification     #851      +/-   ##
=============================================================
+ Coverage                      82.62%   82.64%   +0.01%     
=============================================================
  Files                            137      137              
  Lines                           4501     4511      +10     
  Branches                         587      591       +4     
=============================================================
+ Hits                            3719     3728       +9     
- Misses                           562      563       +1     
  Partials                         220      220              
Impacted Files Coverage Δ
...in/java/com/revenuecat/purchases/common/Backend.kt 85.25% <0.00%> (-0.40%) ⬇️
...om/revenuecat/purchases/strings/IdentityStrings.kt 0.00% <ø> (ø)
...java/com/revenuecat/purchases/common/HTTPClient.kt 88.80% <100.00%> (ø)
...m/revenuecat/purchases/identity/IdentityManager.kt 97.18% <100.00%> (+0.40%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tonidero tonidero force-pushed the toniricodiez/sdk-2897-requestdate-not-returning-updated-value branch from 44c9010 to 60859d1 Compare March 8, 2023 12:01
@tonidero tonidero force-pushed the toniricodiez/sdk-2895-create-a-grace-period-for-entitlements branch from 45f36c4 to 817c4fd Compare March 8, 2023 12:02

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nice

Comment thread test-utils/src/main/java/com/revenuecat/purchases/utils/Responses.kt Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏🏻

Base automatically changed from toniricodiez/sdk-2897-requestdate-not-returning-updated-value to entitlements-verification March 8, 2023 15:25
@tonidero tonidero force-pushed the toniricodiez/sdk-2895-create-a-grace-period-for-entitlements branch from a962240 to 282fd0b Compare March 8, 2023 15:40
@tonidero tonidero merged commit 2c755dc into entitlements-verification Mar 8, 2023
@tonidero tonidero deleted the toniricodiez/sdk-2895-create-a-grace-period-for-entitlements branch March 8, 2023 15:59
tonidero added a commit that referenced this pull request Mar 15, 2023
### Description
Completes
[SDK-2895](https://linear.app/revenuecat/issue/SDK-2895/create-a-grace-period-for-entitlements-when-the-backend-is-offline).

Android equivalent of:
RevenueCat/purchases-ios#2288

In this PR we are adding a 3 day grace period to entitlements expiration
dates.
@tonidero tonidero mentioned this pull request Mar 15, 2023
@tonidero tonidero mentioned this pull request Mar 31, 2023
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