Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: suspend on RECONCILING, mem provider fixes #796

Merged
merged 4 commits into from
Mar 1, 2024

Conversation

toddbaert
Copy link
Member

@toddbaert toddbaert commented Jan 30, 2024

This PR adds "suspend on reconciling" to the React SDK, in a manner consistent with this spec.

When the SDK emits a PROVIDER_RECONCILING event, loaders are displayed until the CONTEXT_CHANGED event is emitted.

Copy link
Member

@lukas-reining lukas-reining left a comment

Choose a reason for hiding this comment

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

LGTM!

@toddbaert toddbaert marked this pull request as draft February 1, 2024 15:46
@toddbaert toddbaert force-pushed the feat/suspend-on-stale branch 2 times, most recently from f17531a to 738d6ac Compare March 1, 2024 19:55
@toddbaert toddbaert changed the title feat: suspend on STALE, mem provider fixes feat: suspend on RECONCILING, mem provider fixes Mar 1, 2024
* Adds suspend on STALE while context changes
* Improves event semantics for in-memory provider

Signed-off-by: Todd Baert <[email protected]>
Signed-off-by: Todd Baert <[email protected]>
@toddbaert toddbaert marked this pull request as ready for review March 1, 2024 21:45
@toddbaert toddbaert added this pull request to the merge queue Mar 1, 2024
Merged via the queue into main with commit 8101ff1 Mar 1, 2024
10 checks passed
@toddbaert toddbaert deleted the feat/suspend-on-stale branch March 1, 2024 21:51
github-merge-queue bot pushed a commit that referenced this pull request Mar 5, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.4.14](web-sdk-v0.4.13...web-sdk-v0.4.14)
(2024-03-05)


### ✨ New Features

* maintain state in SDK, add RECONCILING
([#795](#795))
([cfb0a69](cfb0a69))
* suspend on RECONCILING, mem provider fixes
([#796](#796))
([8101ff1](8101ff1))


### 🐛 Bug Fixes

* allow iteration over all event types
([#844](#844))
([411c7b4](411c7b4))
* correct rollup to bundle all but core
([#846](#846))
([f451e25](f451e25))


### 🧹 Chore

* **main:** release core 0.0.27
([#839](#839))
([ccbb1f9](ccbb1f9))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Signed-off-by: OpenFeature Bot <[email protected]>
Signed-off-by: Todd Baert <[email protected]>
Co-authored-by: Todd Baert <[email protected]>
toddbaert pushed a commit that referenced this pull request Mar 5, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.2.1-experimental](react-sdk-v0.2.0-experimental...react-sdk-v0.2.1-experimental)
(2024-03-05)


### ✨ New Features

* maintain state in SDK, add RECONCILING
([#795](#795))
([cfb0a69](cfb0a69))
* suspend on RECONCILING, mem provider fixes
([#796](#796))
([8101ff1](8101ff1))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Signed-off-by: OpenFeature Bot <[email protected]>
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.

4 participants