Skip to content

Fix promotional offer not being displayed#2173

Merged
vegaro merged 1 commit into
mainfrom
fix-loaded
Feb 15, 2025
Merged

Fix promotional offer not being displayed#2173
vegaro merged 1 commit into
mainfrom
fix-loaded

Conversation

@vegaro

@vegaro vegaro commented Feb 14, 2025

Copy link
Copy Markdown
Member

We were not returning the updated state when loading the promotional offer, so it was effectively never been set in the state

}
}
return false
return loaded

@vegaro vegaro Feb 14, 2025

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think I can refactor this to check if there's a promotionalOfferData in the state in the calling function, but I would need to do more tests

@tonidero tonidero left a comment

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.

:shipit:

@vegaro vegaro enabled auto-merge (squash) February 15, 2025 08:29
@vegaro vegaro merged commit 2c13375 into main Feb 15, 2025
@vegaro vegaro deleted the fix-loaded branch February 15, 2025 08:50
@codecov

codecov Bot commented Feb 15, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.98%. Comparing base (4a0ff72) to head (def68c4).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2173   +/-   ##
=======================================
  Coverage   80.98%   80.98%           
=======================================
  Files         276      276           
  Lines        9388     9388           
  Branches     1333     1333           
=======================================
  Hits         7603     7603           
  Misses       1226     1226           
  Partials      559      559           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

vegaro pushed a commit that referenced this pull request Feb 15, 2025
**This is an automatic release.**

## RevenueCatUI SDK
### Customer Center
#### 🐞 Bugfixes
* Fix promotional offer not being displayed (#2173) via Cesar de la Vega
(@vegaro)
* Fix back button dismissing the Customer Center (#2171) via Cesar de la
Vega (@vegaro)

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants