feat(paywalls): fall back to the offerings paywall when the workflow fetch fails#3709
Draft
vegaro wants to merge 1 commit into
Draft
Conversation
2 tasks
Member
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
2 tasks
7ec5612 to
2db6417
Compare
862b192 to
a9d8b65
Compare
2db6417 to
a0633cb
Compare
This was referenced Jul 6, 2026
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## cesar/workflows-config-endpoint-rewrite #3709 +/- ##
========================================================================
Coverage 80.32% 80.32%
========================================================================
Files 405 405
Lines 16576 16576
Branches 2377 2377
========================================================================
Hits 13314 13314
Misses 2315 2315
Partials 947 947 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
a9d8b65 to
76c654f
Compare
a0633cb to
f45cbe3
Compare
76c654f to
321280b
Compare
f45cbe3 to
78f1020
Compare
321280b to
014ac68
Compare
78f1020 to
2398df9
Compare
014ac68 to
162b48a
Compare
2398df9 to
2874393
Compare
162b48a to
9503ffc
Compare
2874393 to
55ec8a7
Compare
55ec8a7 to
ebbd507
Compare
9503ffc to
516b530
Compare
0087da7 to
9ac153f
Compare
ebbd507 to
526f90c
Compare
…fetch fails The /v1/config endpoint can be disabled for the session after a 4xx (the kill switch), be unreachable, or simply have no workflow for the offering yet. In all of those cases the offering still carries the paywall that /offerings delivered, so render that through the regular components path instead of surfacing PaywallState.Error — apps keep showing a paywall when the config layer can't serve one. The error state remains for offerings with nothing to fall back to. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
526f90c to
d41a4f2
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
/offeringsdelivered, so render that through the regular components path instead of surfacingPaywallState.Error.Stacked on #3716.
Test plan
:ui:revenuecatui:testDefaultsBc8DebugUnitTest