Skip to content

Conversation

ikerib
Copy link
Contributor

@ikerib ikerib commented Jan 7, 2025

Brevo third-party service now supports webhooks.

Related issue #20514

Brevo third-party service now supports webhooks.
@carsonbot carsonbot added this to the 7.3 milestone Jan 7, 2025
@carsonbot carsonbot changed the title Update notifier.rst [Notifier] Update notifier.rst Jan 7, 2025
@javiereguiluz javiereguiluz merged commit b2a3db2 into symfony:7.3 Jan 7, 2025
3 checks passed
@javiereguiluz
Copy link
Member

Thanks Iker ... and congrats on your first Symfony Docs contribution 🎉

@xabbuh
Copy link
Member

xabbuh commented Jan 7, 2025

Does the Brevo bridge really support webhooks now?

@ikerib
Copy link
Contributor Author

ikerib commented Jan 7, 2025

Does the Brevo bridge really support webhooks now?

The had webhooks for emails, I used it for SMS and it works.
After that, I asked to Brevo devolpers and they told me that is ok. I copy & paste their answer:

In this case, I inform you that the webhook you have used is more for emails. However, our technical team has informed me that when using our API to send SMS messages, you can add your callback URL in the webUrl field. Then you will receive all events related to the SMS message on the webhook.
 
Please see below: (https://developers.sendinblue.com/reference/sendtransacsms)

@xabbuh
Copy link
Member

xabbuh commented Jan 7, 2025

If I understand it correctly, there’s two different things. One is if Brevo supports webhooks in general. But the table we have in the docs as far as I understand means whether or not we have support for them through the Symfony Webhook component which doesn’t seem to be the case if I don’t miss anything.

@ikerib
Copy link
Contributor Author

ikerib commented Jan 8, 2025

You are right. I misunderstood the table.
The PR I made enables the ability to specify a url for webhoook, not symfony webhook compatibility.

javiereguiluz added a commit that referenced this pull request Jan 9, 2025
… bridge (xabbuh)

This PR was merged into the 7.3 branch.

Discussion
----------

[Notifier] revert webhook support for the Brevo Notifier bridge

reverts #20539 (see the discussion)

Commits
-------

ed80f8b revert webhook support for the Brevo Notifier bridge
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