xstate icon indicating copy to clipboard operation
xstate copied to clipboard

Add dev-time warning for invoke ID confusion

Open davidkpiano opened this issue 3 years ago • 0 comments

Makes sense, thanks for letting us know about the fix @Andarist and @davidkpiano .

Just my two cents because I have no idea what the original bug was - perhaps it makes sense to adjust the type and make this value required if it could potentially cause an exception if the value's not present. I understand that it's not required for most typical use cases though, so maybe we could default to using a different value (src?) as the id, and report a dev-time warning if there's a collision.

Feel free to close this issue since it's not an issue with the package itself. Thanks!

Originally posted by @rtmruczek in https://github.com/statelyai/xstate/issues/2985#issuecomment-1025889017

davidkpiano avatar Feb 01 '22 14:02 davidkpiano