Skip to content

[Notifier] Add TurboSms Bridge #15403

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
Aug 4, 2021

Conversation

fre5h
Copy link
Contributor

@fre5h fre5h commented Jun 2, 2021

TurboSms Bridge: symfony/symfony#41522

@wouterj wouterj modified the milestones: next, 5.4 Jun 17, 2021
OskarStark added a commit to symfony/symfony that referenced this pull request Aug 4, 2021
This PR was squashed before being merged into the 5.4 branch.

Discussion
----------

[Notifier] Add TurboSms Bridge

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| License       | MIT
| Doc PR        | symfony/symfony-docs#15403
| Recipe PR       | symfony/recipes#953

Add TurboSms bridge to Symfony Notifier

https://turbosms.ua/api.html This service is very popular in Ukraine

Commits
-------

12bacf4 [Notifier] Add TurboSms Bridge
@OskarStark OskarStark removed the Waiting Code Merge Docs for features pending to be merged label Aug 4, 2021
@OskarStark
Copy link
Contributor

Can you please rebase, so we can merge this PR?
Thanks

@OskarStark OskarStark modified the milestones: next, 5.4 Aug 4, 2021
@fre5h fre5h force-pushed the feature-notifier-turbosms branch from 3a6921c to 10cced4 Compare August 4, 2021 15:05
@fre5h
Copy link
Contributor Author

fre5h commented Aug 4, 2021

@OskarStark rebased

@OskarStark
Copy link
Contributor

It took time, but here we go, this is in now. Thank you very much Artem.

@OskarStark OskarStark merged commit 4c6b780 into symfony:5.4 Aug 4, 2021
@fre5h fre5h deleted the feature-notifier-turbosms branch August 5, 2021 06:21
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