Skip to content

feat: suspend on RECONCILING, mem provider fixes#796

Merged
toddbaert merged 4 commits intomainfrom
feat/suspend-on-stale
Mar 1, 2024
Merged

feat: suspend on RECONCILING, mem provider fixes#796
toddbaert merged 4 commits intomainfrom
feat/suspend-on-stale

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.

@toddbaert toddbaert requested a review from beeme1mr January 30, 2024 17:22
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 <todd.baert@dynatrace.com>
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
@toddbaert toddbaert force-pushed the feat/suspend-on-stale branch from 6aac755 to 9d7fd6c Compare March 1, 2024 21:43
@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
@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 <109696520+openfeaturebot@users.noreply.github.com>
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
Co-authored-by: Todd Baert <todd.baert@dynatrace.com>
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 <109696520+openfeaturebot@users.noreply.github.com>
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