Skip to content

[Messenger] update events for 4.4#12586

Merged
wouterj merged 1 commit intosymfony:4.4from
Tobion:patch-3
Nov 9, 2019
Merged

[Messenger] update events for 4.4#12586
wouterj merged 1 commit intosymfony:4.4from
Tobion:patch-3

Conversation

@Tobion
Copy link
Copy Markdown
Contributor

@Tobion Tobion commented Nov 3, 2019

Documentation for symfony/symfony#34217

Copy link
Copy Markdown
Contributor

@dbrumann dbrumann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor nitpick, but I would prefer if these where ordered alphabetically.

@Tobion
Copy link
Copy Markdown
Contributor Author

Tobion commented Nov 4, 2019

I think listing them by the order they are dispatched is more readable and logical.

@wouterj
Copy link
Copy Markdown
Member

wouterj commented Nov 9, 2019

Thanks! I agree with ordering them in the order they are dispatched.

wouterj added a commit that referenced this pull request Nov 9, 2019
This PR was merged into the 4.4 branch.

Discussion
----------

[Messenger] update events for 4.4

Documentation for symfony/symfony#34217

Commits
-------

3374261 [Messenger] update events for 4.4
@wouterj wouterj merged commit 3374261 into symfony:4.4 Nov 9, 2019
@Tobion Tobion deleted the patch-3 branch November 9, 2019 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants