Skip to content

feat(RevenueCatUI): evaluate state conditions in the override resolver (PWENG-57)#3661

Draft
AlvaroBrey wants to merge 1 commit into
alvarobrey/pweng-57-state-storefrom
alvarobrey/pweng-57-resolver
Draft

feat(RevenueCatUI): evaluate state conditions in the override resolver (PWENG-57)#3661
AlvaroBrey wants to merge 1 commit into
alvarobrey/pweng-57-state-storefrom
alvarobrey/pweng-57-resolver

Conversation

@AlvaroBrey

@AlvaroBrey AlvaroBrey commented Jun 26, 2026

Copy link
Copy Markdown
Contributor
  • Evaluates state_condition in the override resolver, with type-aware comparison.
  • Missing key falls back to the declared default. Undeclared key never applies the override.
  • Component call sites still pass empty state for now. Wiring the live store through to them comes later with the first component (Tab).

AlvaroBrey commented Jun 26, 2026

Copy link
Copy Markdown
Contributor 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.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@AlvaroBrey AlvaroBrey self-assigned this Jun 26, 2026
@AlvaroBrey AlvaroBrey changed the title feat(RevenueCatUI): evaluate state conditions in the override resolver feat(RevenueCatUI): evaluate state conditions in the override resolver (PWENG-57) Jun 26, 2026
@AlvaroBrey

Copy link
Copy Markdown
Contributor Author

@cursor review

@cursor cursor Bot 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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 5500d42. Configure here.

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.

1 participant