Skip to content

[Notifier] Introduce NullMessage and remove transport setter in MessageInterface #38474

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
Merged
merged 1 commit into from
Oct 8, 2020
Merged

[Notifier] Introduce NullMessage and remove transport setter in MessageInterface #38474

merged 1 commit into from
Oct 8, 2020

Conversation

jschaedl
Copy link
Contributor

@jschaedl jschaedl commented Oct 7, 2020

Q A
Branch? 5.x
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Follow-up PR of #36479

@jschaedl jschaedl force-pushed the notifier-webprofiler-panel-follow-up branch from 8e75f74 to d2bc943 Compare October 7, 2020 20:04
@jschaedl jschaedl mentioned this pull request Oct 7, 2020
Merged
@jschaedl jschaedl marked this pull request as ready for review October 7, 2020 20:07
@jschaedl jschaedl force-pushed the notifier-webprofiler-panel-follow-up branch from d2bc943 to 8715076 Compare October 7, 2020 20:08
@jschaedl jschaedl force-pushed the notifier-webprofiler-panel-follow-up branch from 8715076 to 5701e89 Compare October 7, 2020 20:10
@jschaedl jschaedl changed the title Introduce NullMessage and remove transport setter in MessageInterface [Notifier] Introduce NullMessage and remove transport setter in MessageInterface Oct 7, 2020
@fabpot
Copy link
Member

fabpot commented Oct 8, 2020

Thank you @jschaedl.

@fabpot fabpot merged commit 0137609 into symfony:5.x Oct 8, 2020
@xabbuh xabbuh added the Notifier label Oct 8, 2020
@xabbuh xabbuh added this to the 5.2 milestone Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabpot fabpot fabpot approved these changes

Assignees
No one assigned
Labels
Notifier Status: Reviewed
Projects
None yet
Milestone
5.2
Development

Successfully merging this pull request may close these issues.
4 participants
@jschaedl @fabpot @xabbuh @carsonbot