-
Notifications
You must be signed in to change notification settings - Fork 24
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
add redux for handling data-layer ws (re)connection #644
Conversation
aea68df
to
8b9722d
Compare
@@ -12,9 +12,9 @@ export async function consumeAllMessagesInto( | |||
} | |||
} | |||
} catch (err) { | |||
debugger | |||
// debugger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: handle this reconnection so we can continue sending/receiving messages
Deploying with Cloudflare Pages
|
Test this pull request
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #644 +/- ##
==========================================
+ Coverage 74.07% 74.30% +0.23%
==========================================
Files 276 285 +9
Lines 9293 9447 +154
Branches 1260 1265 +5
==========================================
+ Hits 6884 7020 +136
- Misses 2292 2309 +17
- Partials 117 118 +1
☔ View full report in Codecov by Sentry. |
5577f45
to
0ce2e19
Compare
0ce2e19
to
ce5a964
Compare
Removed dataLayer from SDKContextValue. Now it lives in the store.
Reconnecting
Disconnected. No retries after 5 times