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

[Workflow] Removed note about flushing in entered event #12413

Merged
merged 1 commit into from Oct 3, 2019
Merged

[Workflow] Removed note about flushing in entered event #12413

merged 1 commit into from Oct 3, 2019

Commits on Oct 3, 2019

  1. [Workflow] Removed note about flushing in entered event 

    This sentence is confusing and lead to bad practices:

* It's not a good idea to flush in the model layer
* It can lead to many calls to `flush()` method if the user uses a workflow (with many `to` places after a transition)
* It's not possible to "preview" what a transition do, because everything is flushed in the DB
    @lyrixx
    lyrixx committed Oct 3, 2019
    Copy the full SHA
    d7511a1 View commit details
    Browse the repository at this point in the history