Movatterモバイル変換

[0]ホーム
URL:

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

 Sign up
Appearance settings

[Notifier] Add Mailjet Bridge#15436

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
jnadaud wants to merge1 commit intosymfony:5.4fromjnadaud:5.4
Closed

Conversation

@jnadaud
Copy link

Mailjet notifier bridge:symfony/symfony#41705

@OskarStarkOskarStark added this to the5.4 milestoneJun 14, 2021
@OskarStarkOskarStark added the Waiting Code MergeDocs for features pending to be merged labelJun 14, 2021
@carsonbotcarsonbot modified the milestones:5.4,nextJun 14, 2021
@OskarStark
Copy link
Contributor

It looks likefrom is mandatory, can you please use user and pass of the dsn then, while I would useAuth token aspassword andfrom as user.

Same topic for another bridge:https://github.com/symfony/symfony/pull/41522/files#r644752617

fabpot added a commit to symfony/symfony that referenced this pull requestJul 4, 2021
This PR was merged into the 5.4 branch.Discussion----------[Notifier] add Mailjet SMS bridge| Q             | A| ------------- | ---| Branch?       | 5.4| Bug fix?      | no| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->| License       | MIT| Doc |symfony/symfony-docs#15436Adding Mailjet SMS bridge for Notifier componentCommits-------ee744e7 [Notifier] add Mailjet SMS bridge
@javiereguiluzjaviereguiluz added Status: Reviewed and removed Status: Needs Review Waiting Code MergeDocs for features pending to be merged labelsJul 27, 2021
@javiereguiluz
Copy link
Member

We'll need a rebase here to make it mergeable. Thanks!

@OskarStarkOskarStark modified the milestones:next,5.4Aug 4, 2021
@OskarStarkOskarStark mentioned this pull requestAug 4, 2021
@OskarStark
Copy link
Contributor

I added it in#15584

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@OskarStarkOskarStarkOskarStark left review comments

Assignees

No one assigned

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

4 participants

@jnadaud@OskarStark@javiereguiluz@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp