Skip to content

Entitlement verification: Send updated value in listener when verification result changes#856

Merged
tonidero merged 1 commit into
entitlements-verificationfrom
send-updated-different-verification-values
Mar 14, 2023
Merged

Entitlement verification: Send updated value in listener when verification result changes#856
tonidero merged 1 commit into
entitlements-verificationfrom
send-updated-different-verification-values

Conversation

@tonidero

@tonidero tonidero commented Mar 9, 2023

Copy link
Copy Markdown
Contributor

Description

We were not sending updated values in the listener when we only detected a change in the verification result. This fixes that by adding this new property to the equals calculation.

@tonidero tonidero requested a review from a team March 9, 2023 11:11
@codecov

codecov Bot commented Mar 9, 2023

Copy link
Copy Markdown

Codecov Report

Merging #856 (68e1318) into entitlements-verification (89a0e58) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@                    Coverage Diff                     @@
##           entitlements-verification     #856   +/-   ##
==========================================================
  Coverage                      83.28%   83.28%           
==========================================================
  Files                            139      139           
  Lines                           4546     4548    +2     
  Branches                         600      602    +2     
==========================================================
+ Hits                            3786     3788    +2     
  Misses                           540      540           
  Partials                         220      220           
Impacted Files Coverage Δ
...n/java/com/revenuecat/purchases/EntitlementInfo.kt 48.91% <100.00%> (+0.56%) ⬆️
.../java/com/revenuecat/purchases/EntitlementInfos.kt 81.25% <100.00%> (+1.25%) ⬆️

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

@tonidero tonidero merged commit e8247ad into entitlements-verification Mar 14, 2023
@tonidero tonidero deleted the send-updated-different-verification-values branch March 14, 2023 09:02
tonidero added a commit that referenced this pull request Mar 15, 2023
…ation result changes (#856)

### Description
We were not sending updated values in the listener when we only detected
a change in the verification result. This fixes that by adding this new
property to the equals calculation.
@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