statelyai / xstate-viz

Visualizer for XState machines
https://stately.ai/viz
MIT License
434 stars 102 forks source link

Fix simulating events on child actors. #326 #348 #347

Open MarianPalkus opened 2 years ago

MarianPalkus commented 2 years ago

This PR fixes #326 by searching for child actors when simulating events:

changeset-bot[bot] commented 2 years ago

⚠️ No Changeset found

Latest commit: a4d99919b77f812fd43fb34a76e9950ee2de5e09

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

vercel[bot] commented 2 years ago

@MarianPalkus is attempting to deploy a commit to the Stately Team on Vercel.

A member of the Team first needs to authorize it.

MarianPalkus commented 2 years ago

Might be related to #352.

Andarist commented 2 years ago

Could you add an e2e test case for this? It would be way easier to review this with such test case in place

vercel[bot] commented 2 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/statelyai/xstate-viz/92v7KSAVXHzZ9qNA9TgkeUD91x4p
✅ Preview: https://xstate-viz-git-fork-marianpalkus-326fixsimulat-1a4f77-statelyai.vercel.app

MarianPalkus commented 2 years ago

Sure, added a test case. I hope it makes sense.

MarianPalkus commented 2 years ago

Might be related #348.