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.

Sign up for GitHub

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

Jump to bottom

=act #13970 Send transistion on goto(CurrentState) in FSM #15561

Merged
merged 1 commit into from
Jul 19, 2014
Merged

=act #13970 Send transistion on goto(CurrentState) in FSM #15561

merged 1 commit into from
Jul 19, 2014

Conversation

ktoso
Copy link
Member

@ktoso ktoso commented Jul 18, 2014

Squashed and fixed typos in PR prepared by @bambuchaAdm in #15360
Resolves #13970

Thanks Łukasz!

When in `A`:
* `goto(A)` will trigger `onTransition(A -> A)`
* `stay()` will NOT trigger `onTransition`

Includes:
* migration guide
* docs updates
* test

@ktoso ktoso added validating and removed tested labels Jul 18, 2014
This was referenced Jul 18, 2014
Closed
Closed
@ktoso ktoso removed tested labels Jul 18, 2014
@akka-ci
Copy link

akka-ci commented Jul 18, 2014

Pull request validation: FAILED 👎
Refer to this link for build results: https://jenkins.akka.io/job/pr-validator-per-commit-jenkins/371/

@ktoso
Copy link
Member Author

ktoso commented Jul 18, 2014

PLS BUILD

@ktoso ktoso added tested and removed tested labels Jul 18, 2014
@akka-ci
Copy link

akka-ci commented Jul 18, 2014

Pull request validation: SUCCESS 👍
Refer to this link for build results: https://jenkins.akka.io/job/pr-validator-per-commit-jenkins/374/

@ktoso
=act akka#13970 Send transistion on goto(CurrentState) in FSM
2c88bb1 
When in `A`:
* `goto(A)` will trigger `onTransition(A -> A)`
* `stay()` will NOT trigger `onTransition`

Includes:
* migration guide
* docs updates
* test
@ktoso ktoso added validating and removed tested labels Jul 18, 2014
@akka-ci
Copy link

akka-ci commented Jul 18, 2014

Pull request validation: SUCCESS 👍
Refer to this link for build results: https://jenkins.akka.io/job/pr-validator-per-commit-jenkins/378/

ktoso added a commit that referenced this pull request Jul 19, 2014
@ktoso
Merge pull request #15561 from ktoso/master
cb05725 
=act #13970 Send transistion on goto(CurrentState) in FSM
@ktoso ktoso merged commit cb05725 into akka:master Jul 19, 2014
@ktoso ktoso mentioned this pull request Jul 9, 2015
Closed
@alexvaluyskiy alexvaluyskiy mentioned this pull request Mar 16, 2017
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
tested PR that was successfully built and tested by Jenkins
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

FSM: fire onTransition() hooks for certain same-state transitions
2 participants
@ktoso @akka-ci