-
Notifications
You must be signed in to change notification settings - Fork 38
fix: in EventsController, set NotifyInSync to true to avoid omitting events
#519
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
Conversation
this is more coherent (as not dropping any event) to subscription model
🦋 Changeset detectedLatest commit: 31a73c5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
All contributors have signed the CTA ✍️ ✅ |
|
hey @normanzb 👋 Thanks for this contribution! Can you sign the CTA so we can merge? |
|
I have read the CTA Document and I hereby sign the CTA |
|
@ignaciosantise no problem and thanks for looking into the PR, signed |
|
@normanzb thanks! can you run |
|
@ignaciosantise formated, CI seems all passed |
|



this is more coherent (as not dropping any event due to quick firing of events) to subscription model
fix #518