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

[Messenger] make senders/handlers locator accept envelopes #29052

Merged
merged 1 commit into from Nov 1, 2018

Conversation

Projects
None yet
3 participants
@nicolas-grekas
Member

nicolas-grekas commented Nov 1, 2018

Q A
Branch? 4.2
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

As suggested by @Tobion in #29045 (comment) - works for me also.

@nicolas-grekas nicolas-grekas added this to the 4.2 milestone Nov 1, 2018

@nicolas-grekas nicolas-grekas requested a review from sroze as a code owner Nov 1, 2018

@Tobion

Tobion approved these changes Nov 1, 2018

@Tobion

This comment has been minimized.

Member

Tobion commented Nov 1, 2018

Thank you @nicolas-grekas.

@Tobion Tobion merged commit 9cd88b0 into symfony:master Nov 1, 2018

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
fabbot.io Your code looks good.
Details

Tobion added a commit that referenced this pull request Nov 1, 2018

minor #29052 [Messenger] make senders/handlers locator accept envelop…
…es (nicolas-grekas)

This PR was merged into the 4.2-dev branch.

Discussion
----------

[Messenger] make senders/handlers locator accept envelopes

| Q             | A
| ------------- | ---
| Branch?       | 4.2
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

As suggested by @Tobion in #29045 (comment) - works for me also.

Commits
-------

9cd88b0 [Messenger] make senders/handlers locator accept envelopes

@nicolas-grekas nicolas-grekas deleted the nicolas-grekas:messenger-envelop branch Nov 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment