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

Fix storing next_state in Context #76

Merged
merged 1 commit into from
Jul 28, 2023

Conversation

agrare
Copy link
Member

@agrare agrare commented Jul 28, 2023

The next_state was being saved in the workflow instance but not persisted to the context so if the workflow was reloaded the state was never updated

@agrare agrare requested a review from Fryguy as a code owner July 28, 2023 16:58
@agrare agrare added the bug Something isn't working label Jul 28, 2023
@agrare agrare force-pushed the fix_storing_next_state_in_context branch from 13aa307 to f4fa2b6 Compare July 28, 2023 16:59
@miq-bot
Copy link
Member

miq-bot commented Jul 28, 2023

Checked commit agrare@f4fa2b6 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@Fryguy Fryguy merged commit 77e6459 into ManageIQ:master Jul 28, 2023
4 of 5 checks passed
@Fryguy Fryguy self-assigned this Jul 28, 2023
@agrare agrare deleted the fix_storing_next_state_in_context branch July 28, 2023 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants