Fixed duplicated events issue #303
Conversation
🦋 Changeset detectedLatest commit: 01bb94b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/statelyai/xstate-viz/FpL6zes6kgduz3qGSBpDjq1HoF4w |
| @@ -68,38 +68,6 @@ export const simulationMachine = simModel.createMachine( | |||
| ? 'inspecting' | |||
| : 'visualizing', | |||
| entry: assign({ notifRef: () => spawn(notifMachine) }), | |||
| invoke: { | |||
There was a problem hiding this comment.
Or is this for visualisations done inside the app during development?
There was a problem hiding this comment.
Does spawning/invoking machines still work? I think that's what this was for.
There was a problem hiding this comment.
Correct, it didn't! Have now made it so it only reports from interpreters that have a parent - i.e. ONLY child machines/invocations
There was a problem hiding this comment.
I'm not familiar with this part but probably @Andarist does
| events: (ctx, e) => { | ||
| return produce(ctx.events, (draft) => { | ||
| draft.push({ | ||
| ...e.state._event, | ||
| timestamp: Date.now(), | ||
| sessionId: e.sessionId, | ||
| }); | ||
| }), | ||
| }); |
There was a problem hiding this comment.
This looks to be an unnecessary change.
There was a problem hiding this comment.
True, it's only formatting though
Co-authored-by: David Khourshid <davidkpiano@gmail.com>
No description provided.