-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Feat/notification-twilio #9648
base: develop
Are you sure you want to change the base?
Feat/notification-twilio #9648
Conversation
|
@420coupe is attempting to deploy a commit to the medusajs Team on Vercel. A member of the Team first needs to authorize it. |
e836cbc
to
150a430
Compare
…er and notificationContent, revert back optional template
150a430
to
169683a
Compare
So here's a working notification-twilio integration. Pre-reqs:
There is also a process to register your Brand, get A2P10DC verified, and finally submit for campaign approval for your use case in production.
|
Should i create the event webhook with this same PR ? |
Attempting to extend notification service to add twilio as an sms/mms provider. However i am having issues trying to test locally. Carlos has provided the package.json resolutions as an option but it does not seem to be pulling all local changes to my cloned medusa repo.