Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Parameterize Mailgun's region #31998

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
Jun 14, 2019
Merged

Parameterize Mailgun's region #31998

merged 1 commit into from
Jun 14, 2019

Conversation

jderusse
Copy link
Member

@jderusse jderusse commented Jun 11, 2019

Q A
Branch? 4.3
Bug fix? yes
New feature? no
BC breaks? yes
Deprecations? no
Tests pass? yes
Fixed tickets #31810
License MIT
Doc PR TODO

Mailgun is available in 2 regions (US and EU), when registering a custom domain, users can choose one of the 2 regions and have to use the right the endpoint (see documentation https://documentation.mailgun.com/en/latest/api-intro.html?highlight=smtp.mailgun.org#mailgun-regions).

This PR make the endpoint/region configurable.

@Koc
Copy link
Contributor

Koc commented Jun 11, 2019

possible duplicate of #31897
relates to #31810

@jderusse
Copy link
Member Author

jderusse commented Jun 11, 2019

my bad. Implemented the code last week and didn't noticed someone opened another PR

@jderusse jderusse closed this Jun 11, 2019
@jderusse
Copy link
Member Author

jderusse commented Jun 11, 2019

looking at #31897, I think that the approach of this PR is way simpler. Reopening it

@jderusse jderusse reopened this Jun 11, 2019
@jderusse jderusse force-pushed the fix-mailgun-host branch 3 times, most recently from bdd7b45 to 35aac7f Compare June 11, 2019 21:59
@nicolas-grekas nicolas-grekas added this to the next milestone Jun 12, 2019
@jderusse jderusse force-pushed the fix-mailgun-host branch 2 times, most recently from 762ab8c to 85cd694 Compare June 12, 2019 08:11
@jderusse jderusse changed the base branch from 4.4 to 4.3 June 13, 2019 17:53
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.3 Jun 13, 2019
@fabpot
Copy link
Member

fabpot commented Jun 14, 2019

Thank you @jderusse.

@fabpot fabpot merged commit 7439c8d into symfony:4.3 Jun 14, 2019
fabpot added a commit that referenced this pull request Jun 14, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

Parameterize Mailgun's region

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | yes
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #31810
| License       | MIT
| Doc PR        | TODO

Mailgun is available in 2 regions (US and EU), when registering a custom domain, users can choose one of the 2 regions and **have to** use the right the endpoint (see documentation https://documentation.mailgun.com/en/latest/api-intro.html?highlight=smtp.mailgun.org#mailgun-regions).

This PR make the endpoint/region configurable.

Commits
-------

7439c8d Parameterize Mailgun's region
@fabpot fabpot mentioned this pull request Jun 26, 2019
@jderusse jderusse deleted the fix-mailgun-host branch August 2, 2019 12:15
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.

6 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.