Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[FrameworkBundle] Add missing webhook parsers #53004

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

Merged
merged 4 commits into from
Dec 13, 2023

Conversation

alexandre-daubois
Copy link
Member

@alexandre-daubois alexandre-daubois commented Dec 12, 2023

Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues -
License MIT

Detected when reviewing this documentation PR: symfony/symfony-docs#19268

Should be merged after #53007, which brings notifier services removal conditionally

@alexandre-daubois alexandre-daubois force-pushed the add-missing-webhooks-parsers branch from 6a62278 to 86cc1db Compare December 12, 2023 11:55
@alexandre-daubois alexandre-daubois changed the base branch from 7.1 to 6.4 December 12, 2023 11:56
@alexandre-daubois alexandre-daubois force-pushed the add-missing-webhooks-parsers branch from 86cc1db to f851a39 Compare December 12, 2023 12:04
@TimoBakx
Copy link
Member

Nice work!

…l and add notifier parser service removal (alexandre-daubois)

This PR was merged into the 6.3 branch.

Discussion
----------

[FrameworkBundle] Fix webhook parser service removal and add notifier parser service removal

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

Should be merged before symfony#53004

Commits
-------

b7a17ad [FrameworkBundle] Fix webhook parser service removal and add notifier parser service removal
* 6.3:
  [FrameworkBundle] Fix webhook parser service removal and add notifier parser service removal
@fabpot fabpot modified the milestones: 7.1, 6.4 Dec 13, 2023
@fabpot fabpot force-pushed the add-missing-webhooks-parsers branch from f851a39 to b52b9e1 Compare December 13, 2023 07:11
@fabpot
Copy link
Member

fabpot commented Dec 13, 2023

Thank you @alexandre-daubois.

@fabpot fabpot merged commit 877b173 into symfony:6.4 Dec 13, 2023
This was referenced Dec 30, 2023
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.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.