Skip to content

Ensure that from_state is set after successfully firing an event. - #826

Open
KirtashW17 wants to merge 1 commit into
aasm:masterfrom
KirtashW17:803_aasm_from_state_not_always_populated
Open

KirtashW17 wants to merge 1 commit into
aasm:masterfrom
KirtashW17:803_aasm_from_state_not_always_populated

Conversation

@KirtashW17

Copy link
Copy Markdown

Fixes #803.

I'm not sure is I'm assigning from_state in the right place. The drawback is that we are assigning it twice when the transition does include a from clause.

@qlty-cloud-legacy

Copy link
Copy Markdown

Code Climate has analyzed commit 2f24768 and detected 0 issues on this pull request.

View more on Code Climate.

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

Successfully merging this pull request may close these issues.

aasm.from_state is not always populated

1 participant