Skip to content

[Mailer] Add new email testing functions #18269

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 12, 2023
Merged

Conversation

JohJohan
Copy link
Contributor

@JohJohan JohJohan commented May 1, 2023

@JohJohan
Copy link
Contributor Author

JohJohan commented May 2, 2023

I updated target branch to 6.3 and rebased.

@OskarStark OskarStark changed the title [Mailer] Add new email testing functions (fixes #50200) [Mailer] Add new email testing functions May 2, 2023
Copy link
Contributor

@ndench ndench left a comment

Choose a reason for hiding this comment

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

Thanks!

@OskarStark
Copy link
Contributor

Please target 6.4 branch, thanks

@JohJohan JohJohan changed the base branch from 6.3 to 6.4 June 12, 2023 07:43
fabpot added a commit to symfony/symfony that referenced this pull request Jul 9, 2023
…EmailSubjectNotContains` methods (johanadivare)

This PR was merged into the 6.4 branch.

Discussion
----------

[Mailer] Adds `assertEmailSubjectContains` and `assertEmailSubjectNotContains` methods

> Adds assertEmailSubjectContains and assertEmailSubjectNotContains

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | No
| New feature?  | yes
| Deprecations? | No
| Tickets       | Fix #49939
| License       | MIT
| Doc PR        | symfony/symfony-docs#18269
<!--
Replace this notice by a short README for your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - Bug fixes must be submitted against the lowest maintained branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the latest branch.
 - For new features, provide some code snippets to help understand usage.
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

Commits
-------

55ba7b8 [Mailer] Add EmailSubjectContains constraint (#49939)
symfony-splitter pushed a commit to symfony/framework-bundle that referenced this pull request Jul 9, 2023
…EmailSubjectNotContains` methods (johanadivare)

This PR was merged into the 6.4 branch.

Discussion
----------

[Mailer] Adds `assertEmailSubjectContains` and `assertEmailSubjectNotContains` methods

> Adds assertEmailSubjectContains and assertEmailSubjectNotContains

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | No
| New feature?  | yes
| Deprecations? | No
| Tickets       | Fix #49939
| License       | MIT
| Doc PR        | symfony/symfony-docs#18269
<!--
Replace this notice by a short README for your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - Bug fixes must be submitted against the lowest maintained branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the latest branch.
 - For new features, provide some code snippets to help understand usage.
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

Commits
-------

55ba7b8f22 [Mailer] Add EmailSubjectContains constraint (#49939)
symfony-splitter pushed a commit to symfony/mime that referenced this pull request Jul 9, 2023
…EmailSubjectNotContains` methods (johanadivare)

This PR was merged into the 6.4 branch.

Discussion
----------

[Mailer] Adds `assertEmailSubjectContains` and `assertEmailSubjectNotContains` methods

> Adds assertEmailSubjectContains and assertEmailSubjectNotContains

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | No
| New feature?  | yes
| Deprecations? | No
| Tickets       | Fix #49939
| License       | MIT
| Doc PR        | symfony/symfony-docs#18269
<!--
Replace this notice by a short README for your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - Bug fixes must be submitted against the lowest maintained branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the latest branch.
 - For new features, provide some code snippets to help understand usage.
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

Commits
-------

55ba7b8f22 [Mailer] Add EmailSubjectContains constraint (#49939)
@ghost
Copy link

ghost commented Jul 10, 2023

@OskarStark the code is now merged can this also be merged?

@OskarStark OskarStark removed the Waiting Code Merge Docs for features pending to be merged label Jul 10, 2023
@OskarStark
Copy link
Contributor

I am on holiday, but @javiereguiluz can merge it 👍🏻
Thanks for this contribution 😎🙏🙌

@JohJohan
Copy link
Contributor Author

@OskarStark happy holiday 🍹

@javiereguiluz javiereguiluz merged commit 75c3270 into symfony:6.4 Jul 12, 2023
@javiereguiluz
Copy link
Member

Johan, thanks for contributing this feature and the docs for it.

A minor note: while merging we added the missing versionadded directive (see 1752312)

@JohJohan JohJohan deleted the 50200 branch July 12, 2023 07:22
symfony-splitter pushed a commit to symfony/framework-bundle that referenced this pull request Jul 28, 2023
…EmailSubjectNotContains` methods (johanadivare)

This PR was merged into the 6.4 branch.

Discussion
----------

[Mailer] Adds `assertEmailSubjectContains` and `assertEmailSubjectNotContains` methods

> Adds assertEmailSubjectContains and assertEmailSubjectNotContains

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | No
| New feature?  | yes
| Deprecations? | No
| Tickets       | Fix #49939
| License       | MIT
| Doc PR        | symfony/symfony-docs#18269
<!--
Replace this notice by a short README for your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - Bug fixes must be submitted against the lowest maintained branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the latest branch.
 - For new features, provide some code snippets to help understand usage.
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

Commits
-------

55ba7b8f22 [Mailer] Add EmailSubjectContains constraint (#49939)
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