Skip to content

[Notifier] Add links to push bridges' README #18254

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
Apr 26, 2023
Merged

[Notifier] Add links to push bridges' README #18254

merged 1 commit into from
Apr 26, 2023

Conversation

mikaelkael
Copy link
Contributor

As requested in #18253

@carsonbot carsonbot added this to the 5.4 milestone Apr 25, 2023
@OskarStark
Copy link
Contributor

Thank you Mickael.

@OskarStark OskarStark merged commit a49c293 into symfony:5.4 Apr 26, 2023
@OskarStark
Copy link
Contributor

OskarStark commented Apr 26, 2023 •
edited
Loading

Thanks I upmerged this PR to 6.2 and 6.3, are you open to create another one against 6.2, I upmerge, and then against 6.3? Many thanks

This was referenced Apr 26, 2023
Merged
Merged
@mikaelkael
Copy link
Contributor Author

@OskarStark : and voilà!
For 6.2 : #18256
For 6.3 : #18257

OskarStark added a commit that referenced this pull request Apr 26, 2023
@OskarStark
minor #18256 [Notifier] Add links to push bridges' README (mikaelkael)
a34ced5 
This PR was merged into the 6.2 branch.

Discussion
----------

[Notifier] Add links to push bridges' README

Following #18254 and fix merge issue added by f6034cd

Commits
-------

a275650 [Notifier] Add links to push bridges' README
OskarStark added a commit that referenced this pull request Apr 26, 2023
@OskarStark
minor #18257 [Notifier] Add links to push bridges' README (mikaelkael)
89093c5 
This PR was squashed before being merged into the 6.3 branch.

Discussion
----------

[Notifier] Add links to push bridges' README

Following #18254 and fix merge issue added by f6034cd

Commits
-------

0f65b2c [Notifier] Add links to push bridges' README
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alamirault alamirault alamirault approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@mikaelkael @OskarStark @alamirault @carsonbot