Skip to content

[Mailer] Rename Sendinlue to Brevo #18359

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 1 commit into from
Jul 9, 2023
Merged

[Mailer] Rename Sendinlue to Brevo #18359

merged 1 commit into from
Jul 9, 2023

Conversation

blaugueux
Copy link
Contributor

@blaugueux blaugueux commented Jun 1, 2023

Sendinblue mailer transport was deprecated in Symfony 6.4, new transport Brevo was created due to the service name change.

https://www.brevo.com/fr/blog/pourquoi-brevo/

PR : symfony/symfony#50302

javiereguiluz added a commit that referenced this pull request Jun 8, 2023
This PR was submitted for the 6.3 branch but it was merged into the 6.2 branch instead.

Discussion
----------

[Mailer] Improve DSN formats table

This PR try to improve readibility of DSN formats table which displays horizontally too long and under the right sidebar on small screens when reading from [symfony.com](https://symfony.com/doc/current/mailer.html#using-a-3rd-party-transport).

I changed table layout so it displays only two columns instead of four (see captures below).

Please keep in mind that there is a pending PR for renaming Sendinblue to Brevo (#18359) so this PR should be updated accordingly.

![broken_layout](https://github.com/symfony/symfony-docs/assets/8760377/3586fced-4fce-41b8-b3e7-934793d2c120)

![fixed_layout](https://github.com/symfony/symfony-docs/assets/8760377/2115137a-8e2a-4160-aca7-57daef2ed425)

Commits
-------

cc46c48 [Mailer] Improve DSN formats table
@javiereguiluz javiereguiluz added Status: Reviewed and removed Waiting Code Merge Docs for features pending to be merged Status: Needs Review labels Jul 9, 2023
@javiereguiluz javiereguiluz modified the milestones: next, 6.4 Jul 9, 2023
@javiereguiluz
Copy link
Member

Benjamin, thanks and congrats on your first Symfony Docs contribution 🎉

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