Workflow table not updated with workflow state

[imegory]

Today, I re-bundled all my gems (in Gemfile), ran the specs, and one spec failed. It appears that the workflow db table was not updated with the new workflow state. I know that the failing spec used to pass about 1-2 months ago. Anyways, I did override load_workflow_state, and persist_workflow_state methods and then the failing spec passed.

FYI: I am on rails 5.2.1 and my test was using workflow 2.0. I tried following the instructions that suggested to use the active-record gem. In my experience that does not work.

Anyways, just sharing my experience.

[geekq]

Thanks @imegory, your feedback on using/bundling the new workflow 2.0 and workflow-activerecord is important!

Can you please provide a minimal example to reproduce the problem?
Or tell, what do you do differently compared to the minimal sample application
https://github.com/geekq/workflow-rails-sample/blob/master/Gemfile#L40-L41
https://github.com/geekq/workflow-rails-sample/blob/master/app/models/article.rb#L4 ?

Tests using workflow-activerecord 4.1.2, workflow 2.0.0 and activerecord 5.2.2 also run successfully:
https://travis-ci.org/geekq/workflow-activerecord/jobs/478725932
Please tell, if you think anything is missing in the persistence test https://github.com/geekq/workflow-activerecord/blob/develop/test/persistence_test.rb

[imegory]

Thank you for your immediate attention. I am going to look at the working example and compare notes with my code, and let you know pretty soon. Thanks, Igal

…

On Fri, Feb 15, 2019 at 3:31 AM Vladimir Dobriakov ***@***.***> wrote: Thanks @imegory <https://github.com/imegory>, you feedback on using/bundling the new workflow 2.0 and workflow-activerecord is important! Can you please provide a minimal example to reproduce the problem? Or tell, what you do differently compared to the minimal sample application https://github.com/geekq/workflow-rails-sample/blob/master/Gemfile#L40-L41 https://github.com/geekq/workflow-rails-sample/blob/master/app/models/article.rb#L4 ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#212 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADdvRivueZk9DN2TpoTFxGH4Qpxh08oIks5vNoyYgaJpZM4a1WqA> .

[geekq]

Closing due inactivity. Please feel free to reopen the issue if you have new information