Skip to content
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

OnInternalTransitioned seems to be missing but is shown in readme #459

Open
kolpav opened this issue Aug 29, 2021 · 0 comments
Open

OnInternalTransitioned seems to be missing but is shown in readme #459

kolpav opened this issue Aug 29, 2021 · 0 comments

Comments

@kolpav
Copy link

kolpav commented Aug 29, 2021

I did bit of git archeology by running grep-all

[alias]
grep-all = !"f() { git rev-list --all | xargs git grep \"$@\"; }; f"

git grep-all OnInternalTransitioned

7ad4f662ae3d3d6de05b9eb0d2dbbaba067a0338:README.md
stateMachine.OnInternalTransitioned((state, trigger) => { });

Which makes me believe that OnInternalTransitioned callback never existed. Is it made up 😄 ? Jokes aside it seems really useful it would be great to have it.

These seems to be all On* methods

image

OnTransitionCompleted is not triggered by internal transitions neither OnTransitioned.

<PackageReference Include="stateless" Version="5.11.0" />

Great library btw 👍

@kolpav kolpav changed the title OnInternalTransitioned seems to be missing OnInternalTransitioned seems to be missing 🤔 Aug 29, 2021
@kolpav kolpav changed the title OnInternalTransitioned seems to be missing 🤔 OnInternalTransitioned seems to be missing Aug 29, 2021
@kolpav kolpav changed the title OnInternalTransitioned seems to be missing OnInternalTransitioned seems to be missing Aug 29, 2021
@kolpav kolpav changed the title OnInternalTransitioned seems to be missing OnInternalTransitioned seems to be missing but shown is shown in readme Aug 29, 2021
@kolpav kolpav changed the title OnInternalTransitioned seems to be missing but shown is shown in readme OnInternalTransitioned seems to be missing but is shown shown in readme Aug 29, 2021
@kolpav kolpav changed the title OnInternalTransitioned seems to be missing but is shown shown in readme OnInternalTransitioned seems to be missing but is shown in readme Aug 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant