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

[Notifier][Prelude] Add bridge #62801

Merged
nicolas-grekas merged 1 commit intosymfony:8.1symfony/symfony:8.1from
zairigimad:feat/prelude-notifier-smszairigimad/symfony:feat/prelude-notifier-smsCopy head branch name to clipboard
Apr 30, 2026
Merged

[Notifier][Prelude] Add bridge #62801
nicolas-grekas merged 1 commit intosymfony:8.1symfony/symfony:8.1from
zairigimad:feat/prelude-notifier-smszairigimad/symfony:feat/prelude-notifier-smsCopy head branch name to clipboard

Conversation

@zairigimad
Copy link
Copy Markdown
Contributor

@zairigimad zairigimad commented Dec 17, 2025 •
edited
Loading

Q A
Branch? 8.1
Bug fix? no
New feature? yes
Deprecations? no
Recipe symfony/recipes#1502
Issues Fix #62802
License MIT

This PR adds the support for Prelude Notification ( Prelude is a provider for OTP, Transactional SMS , and whatsapp messaging)

https://docs.prelude.so/introduction/welcome-to-prelude

@carsonbot carsonbot added this to the 8.1 milestone Dec 17, 2025
@OskarStark OskarStark changed the title [Notifier] Add Prelude Notifier for SMS [Notifier][Prelude] Add bridge Dec 17, 2025
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/.gitattributes Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/CHANGELOG.md Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/CHANGELOG.md Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/LICENSE Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/README.md Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/Tests/PreludeTransportFactoryTest.php Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/composer.json Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/composer.json Outdated
OskarStark
OskarStark reviewed Dec 17, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@OskarStark OskarStark left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Leas update the UnsupportedSchemeException + you can prepare a recipe in symfony/recipes repository, thanks

@zairigimad zairigimad force-pushed the feat/prelude-notifier-sms branch from ac3aed7 to 25486e5 Compare December 17, 2025 13:43
@zairigimad
Copy link
Copy Markdown
Contributor Author

zairigimad commented Dec 17, 2025

UnsupportedSchemeException

I did fix the majority of comments, for UnsupportedSchemeException what should i update ?

I will add the recipe asap

@zairigimad zairigimad force-pushed the feat/prelude-notifier-sms branch 8 times, most recently from 3232eb7 to e439429 Compare December 17, 2025 15:34
@zairigimad
Copy link
Copy Markdown
Contributor Author

zairigimad commented Dec 17, 2025

Leas update the UnsupportedSchemeException + you can prepare a recipe in symfony/recipes repository, thanks

recipe done

nicolas-grekas
nicolas-grekas reviewed Dec 17, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

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

cool, thanks for submitting!

Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/CHANGELOG.md Outdated
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/composer.json Outdated
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/phpunit.xml.dist Outdated
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/PreludeOptions.php Outdated
@zairigimad zairigimad force-pushed the feat/prelude-notifier-sms branch 2 times, most recently from 985aff2 to ec4ead8 Compare December 30, 2025 16:09
@zairigimad
Copy link
Copy Markdown
Contributor Author

zairigimad commented Feb 7, 2026

Hello @OskarStark @nicolas-grekas , i fixed the feedbacks

GromNaN
GromNaN reviewed Mar 18, 2026
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/phpunit.xml.dist Outdated
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/PreludeOptions.php Outdated
@zairigimad zairigimad force-pushed the feat/prelude-notifier-sms branch from ec4ead8 to 9e1303c Compare March 18, 2026 23:18
@zairigimad zairigimad requested review from GromNaN and stof March 18, 2026 23:20
welcoMattic
welcoMattic reviewed Mar 27, 2026
View reviewed changes
Comment thread src/Symfony/Component/Notifier/Bridge/Prelude/LICENSE Outdated
@zairigimad zairigimad force-pushed the feat/prelude-notifier-sms branch from 9e1303c to 7737c65 Compare March 30, 2026 15:43
@nicolas-grekas nicolas-grekas force-pushed the feat/prelude-notifier-sms branch from 7737c65 to 6d5c154 Compare April 30, 2026 08:43
@nicolas-grekas
Copy link
Copy Markdown
Member

nicolas-grekas commented Apr 30, 2026

Thank you @zairigimad.

@nicolas-grekas nicolas-grekas merged commit 2a032cc into symfony:8.1 Apr 30, 2026
5 of 11 checks passed
@nicolas-grekas nicolas-grekas mentioned this pull request Apr 30, 2026
Open
@fabpot fabpot mentioned this pull request May 6, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nicolas-grekas nicolas-grekas nicolas-grekas left review comments

@welcoMattic welcoMattic welcoMattic left review comments

@GromNaN GromNaN GromNaN approved these changes

@OskarStark OskarStark Awaiting requested review from OskarStark OskarStark is a code owner

@stof stof Awaiting requested review from stof

Assignees

No one assigned

Labels

Feature Notifier Status: Reviewed

Projects

None yet

Milestone

8.1

Development

Successfully merging this pull request may close these issues.

[Notifier] Add Prelude Provider Bridge

7 participants

@zairigimad @nicolas-grekas @GromNaN @stof @welcoMattic @OskarStark @carsonbot
Morty Proxy This is a proxified and sanitized view of the page, visit original site.