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: React Native SSE Support #1012

Merged
merged 11 commits into from
Dec 13, 2024
Merged

feat: React Native SSE Support #1012

merged 11 commits into from
Dec 13, 2024

Conversation

jonathannorris
Copy link
Member

@jonathannorris jonathannorris commented Dec 12, 2024

Adding SSE connections to the React-Native SDK to enable live-updates of configs.

  • using react-native-sse library
  • re-factored JS SDK SSE connection to pass in react-native-sse Event Source class from react native lib

Note: will need to mention in the docs the limitations on react native <0.75 requiring flipper to be disabled: binaryminds/react-native-sse#61

Copy link

vercel bot commented Dec 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
js-sdks-web-elements ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 13, 2024 9:02pm
js-sdks-with-provider ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 13, 2024 9:02pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
js-sdks-next-js-page-router ⬜️ Ignored (Inspect) Dec 13, 2024 9:02pm

sdk/js/src/Client.ts Outdated Show resolved Hide resolved
sdk/js/src/Client.ts Outdated Show resolved Hide resolved
"@react-native-async-storage/async-storage": "^1.17.11",
"react-native-device-info": "^8.7.0",
"react-native-get-random-values": "^1.7.2"
"react-native-get-random-values": "^1.7.2",
"react-native-sse": "^1.2.1"
Copy link
Contributor

Choose a reason for hiding this comment

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

note: this will require docs update as we need to install the other packages separately

Copy link
Member Author

Choose a reason for hiding this comment

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

Why wouldn't it be installed from the dependency? seems to work in the E2E test app.

Copy link
Contributor

Choose a reason for hiding this comment

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

im not 100 percent sure, but we have that explicit in the docs and in the past when i have tried to install, required me to add those dependencies have not tested recently tbh

@jonathannorris jonathannorris merged commit 201279b into main Dec 13, 2024
12 checks passed
@jonathannorris jonathannorris deleted the feat-react-native-sse branch December 13, 2024 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants