Skip to content
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

[Notifier][Webhook] Add Vonage support #18745

Closed
fabpot opened this issue Aug 12, 2023 · 4 comments
Closed

[Notifier][Webhook] Add Vonage support #18745

fabpot opened this issue Aug 12, 2023 · 4 comments
Labels
hasPR A Pull Request has already been submitted for this issue. Notifier Webhook
Milestone
6.4

Comments

@fabpot
Copy link
Member

fabpot commented Aug 12, 2023

Q A
Feature PR symfony/symfony#51315
PR author(s) @smnandre
Merged in 6.4

We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.

To fix this issue, please create a PR against the 6.4 branch in the symfony-docs repository.

Thank you! 😃
@fabpot fabpot added this to the 6.4 milestone Aug 12, 2023
@smnandre
Copy link
Contributor

Could i get some help to redact this one ? I could use some advice / example

@OskarStark
Copy link
Contributor

Check the notifier.rst file in this repo and add your bridge to the table 👍🏻😃

@smnandre
Copy link
Contributor

Check the notifier.rst file in this repo and add your bridge to the table 👍🏻😃

The notifier part was already there, i only added the webhook support 😅

@OskarStark
Copy link
Contributor

Oh sorry I missed that.

We currently don't have a Webhook documentation, so let's keep this ob hold

@alexandre-daubois alexandre-daubois mentioned this issue Dec 12, 2023
Merged
OskarStark added a commit that referenced this issue Dec 12, 2023
@OskarStark
feature #19268 [Webhook] Add bridges for Webhook (alexandre-daubois)
9d26845 
This PR was merged into the 6.4 branch.

Discussion
----------

[Webhook] Add bridges for Webhook

Fix #18692, #18694, #18745, #18800

Commits
-------

82ec06e [Webhook] Add missing webhook parsers
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue. Notifier Webhook
Projects
None yet
Milestone
6.4
Development

No branches or pull requests
5 participants
@fabpot @OskarStark @smnandre @xabbuh @carsonbot