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

Should not define initial state as state #29

Closed
jvalkeal opened this issue Mar 22, 2015 · 0 comments · Fixed by #58
Closed

Should not define initial state as state #29

jvalkeal opened this issue Mar 22, 2015 · 0 comments · Fixed by #58
Labels
type/enhancement Is an enhancement request
Milestone
1.0.0.M2

Comments

@jvalkeal
Copy link
Contributor

Currently we need to do this:

@Override
public void configure(StateMachineStateConfigurer<States, Events> states) throws Exception {
  states
    .withStates()
      .initial(States.IDLE)
      .state(States.IDLE)
}

should tweak configurer so that state() is not needed for particular state if it's already introduced with initial().
@jvalkeal jvalkeal added the type/enhancement Is an enhancement request label Mar 22, 2015
@jvalkeal jvalkeal modified the milestone: 1.0.0.M2 Apr 16, 2015
jvalkeal added a commit to jvalkeal/spring-statemachine that referenced this issue May 6, 2015
@jvalkeal
Should not define initial state as state
8e42ca0 
- Tweak DefaultStateConfigurer so that we don't need
  to define initial state as additional state. Also
  fixed some other issue which caused a bit of a mess
  when i.e multiple states were introduces twice.
- Fixes spring-projects#29
@jvalkeal jvalkeal mentioned this issue May 6, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement Is an enhancement request
Projects
None yet
Milestone
1.0.0.M2
Development

Successfully merging a pull request may close this issue.

Should not define initial state as state jvalkeal/spring-statemachine
1 participant
@jvalkeal