Skip to content

Entitlement verification: Rename not verified to not requested and error to failed#838

Merged
tonidero merged 3 commits into
entitlements-verificationfrom
entitlement-verification-rename-not-verified-to-not-requested
Mar 7, 2023
Merged

Entitlement verification: Rename not verified to not requested and error to failed#838
tonidero merged 3 commits into
entitlements-verificationfrom
entitlement-verification-rename-not-verified-to-not-requested

Conversation

@tonidero

@tonidero tonidero commented Mar 3, 2023

Copy link
Copy Markdown
Contributor

Description

Following the conversation in SDK-2896, we are renaming NOT_VERIFIED to NOT_REQUESTED. We're also renaming ERROR to FAILED to match iOS.

@tonidero tonidero requested a review from a team March 3, 2023 10:29
@codecov

codecov Bot commented Mar 3, 2023

Copy link
Copy Markdown

Codecov Report

Merging #838 (1a4c08a) into entitlements-verification (746c8ce) will not change coverage.
The diff coverage is 80.00%.

@@                    Coverage Diff                     @@
##           entitlements-verification     #838   +/-   ##
==========================================================
  Coverage                      82.61%   82.61%           
==========================================================
  Files                            137      137           
  Lines                           4498     4498           
  Branches                         584      584           
==========================================================
  Hits                            3716     3716           
  Misses                           562      562           
  Partials                         220      220           
Impacted Files Coverage Δ
...revenuecat/purchases/common/CustomerInfoFactory.kt 90.00% <0.00%> (ø)
...n/java/com/revenuecat/purchases/EntitlementInfo.kt 48.35% <0.00%> (ø)
.../java/com/revenuecat/purchases/EntitlementInfos.kt 80.00% <0.00%> (ø)
...java/com/revenuecat/purchases/common/HTTPClient.kt 88.63% <100.00%> (ø)
...revenuecat/purchases/common/caching/DeviceCache.kt 95.08% <100.00%> (ø)
...enuecat/purchases/common/networking/ETagManager.kt 97.91% <100.00%> (ø)
...venuecat/purchases/common/networking/HTTPResult.kt 100.00% <100.00%> (ø)
...at/purchases/common/verification/SigningManager.kt 100.00% <100.00%> (ø)
...ava/com/revenuecat/purchases/VerificationResult.kt 100.00% <100.00%> (ø)

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

@tonidero tonidero changed the title Entitlement verification: Rename not verified to not requested Entitlement verification: Rename not verified to not requested and error to failed Mar 3, 2023
@tonidero tonidero merged commit cb9d00a into entitlements-verification Mar 7, 2023
@tonidero tonidero deleted the entitlement-verification-rename-not-verified-to-not-requested branch March 7, 2023 17:28
tonidero added a commit that referenced this pull request Mar 15, 2023
…ror to failed (#838)

### Description
Following the conversation in
[SDK-2896](https://linear.app/revenuecat/issue/SDK-2896/invalidate-devicecache-cache-if-entitlement-verification-is-missing),
we are renaming `NOT_VERIFIED` to `NOT_REQUESTED`. We're also renaming
`ERROR` to `FAILED` to match iOS.
@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