Skip to content

[Mailer] Add Infobip bridge #16760

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 23, 2022
Merged

Conversation

B-Galati
Copy link
Contributor

Hello,

Companion for symfony/symfony#46211!

@carsonbot carsonbot added this to the 6.1 milestone Apr 29, 2022
@OskarStark OskarStark added the Waiting Code Merge Docs for features pending to be merged label Apr 29, 2022
@carsonbot carsonbot modified the milestones: 6.1, next Apr 29, 2022
@B-Galati
Copy link
Contributor Author

B-Galati commented May 2, 2022

@OskarStark thanks a lot, it's updated, should be all clear now ✔️

@OskarStark
Copy link
Contributor

Needs to be against 6.2, as we are in feature freeze now

@B-Galati B-Galati changed the base branch from 6.1 to 6.2 July 18, 2022 11:21
@B-Galati
Copy link
Contributor Author

updated

================== ==============================================

.. versionadded:: 6.2

The ``MailPace`` integration was introduced in Symfony 6.2 (in previous
Symfony versions it was called ``OhMySMTP``).

.. versionadded:: 6.2
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it should be merged with the previous block?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good idea IMO, IIRC we already did the same in the past

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left them separate for now because the existing versionadded is a bit special and explains some renaming. Merging both would look confusing in my opinion. But we can change this at any moment if we disagree. Thanks!

fabpot added a commit to symfony/symfony that referenced this pull request Jul 20, 2022
This PR was merged into the 6.2 branch.

Discussion
----------

[Mailer] Add Infobip bridge

| Q             | A
| ------------- | ---
| Branch?       | 6.2
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| License       | MIT
| Doc PR        | symfony/symfony-docs#16760

Hello,

Add Infobip as new email provider for Symfony Mailer!

Co-authored with @jbdelhommeau 👍

Commits
-------

1f9c282 [Mailer] Add Infobip bridge
symfony-splitter pushed a commit to symfony/framework-bundle that referenced this pull request Jul 20, 2022
This PR was merged into the 6.2 branch.

Discussion
----------

[Mailer] Add Infobip bridge

| Q             | A
| ------------- | ---
| Branch?       | 6.2
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| License       | MIT
| Doc PR        | symfony/symfony-docs#16760

Hello,

Add Infobip as new email provider for Symfony Mailer!

Co-authored with @jbdelhommeau 👍

Commits
-------

1f9c2828f4 [Mailer] Add Infobip bridge
@xabbuh xabbuh removed the Waiting Code Merge Docs for features pending to be merged label Jul 20, 2022
@xabbuh xabbuh modified the milestones: next, 6.2 Jul 20, 2022
@javiereguiluz
Copy link
Member

Thank you Benoit.

@javiereguiluz javiereguiluz merged commit e03b442 into symfony:6.2 Jul 23, 2022
@B-Galati B-Galati deleted the infobip-mailer branch July 26, 2022 09:07
symfony-splitter pushed a commit to symfony/framework-bundle that referenced this pull request Jul 28, 2023
This PR was merged into the 6.2 branch.

Discussion
----------

[Mailer] Add Infobip bridge

| Q             | A
| ------------- | ---
| Branch?       | 6.2
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| License       | MIT
| Doc PR        | symfony/symfony-docs#16760

Hello,

Add Infobip as new email provider for Symfony Mailer!

Co-authored with @jbdelhommeau 👍

Commits
-------

1f9c2828f4 [Mailer] Add Infobip 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.

5 participants