-
Notifications
You must be signed in to change notification settings - Fork 66
Open
Description
🔖 Feature description
The current Vonage SMS adapter uses the SMS API:
messaging/src/Utopia/Messaging/Adapter/SMS/Vonage.php
Lines 48 to 61 in b499c3a
| $result = $this->request( | |
| method: 'POST', | |
| url: 'https://rest.nexmo.com/sms/json', | |
| headers: [ | |
| 'Content-Type' => 'application/x-www-form-urlencoded', | |
| ], | |
| body: [ | |
| 'text' => $message->getContent(), | |
| 'from' => $this->from ?? $message->getFrom(), | |
| 'to' => $to[0], //\implode(',', $to), | |
| 'api_key' => $this->apiKey, | |
| 'api_secret' => $this->apiSecret, | |
| ], | |
| ); |
However, we should add support for the Messages API.
Reference:
🎤 Pitch
The SMS API is limited to SMS and is more expensive than their Messages API1
📝 Tasks
- Create
Utopia\Messaging\Adapter\VonageMessagesBasethat handles the common things between the different message types - Create
Utopia\Messaging\Adapter\SMS\VonageMessagesthat can send SMS messages using the Vonage Messages API
👀 Have you spent some time to check if this issue has been raised before?
- I checked and didn't find similar issue
🏢 Have you read the Code of Conduct?
- I have read the Code of Conduct
Footnotes
Metadata
Metadata
Assignees
Labels
No labels