Skip to content
This repository was archived by the owner on Mar 30, 2026. It is now read-only.

POC - Pinpoint SMS Voice SendTextMessage - #2341

Draft
k-macmillan wants to merge 5 commits into
mainfrom
spike-pool-usage
Draft

POC - Pinpoint SMS Voice SendTextMessage#2341
k-macmillan wants to merge 5 commits into
mainfrom
spike-pool-usage

Conversation

@k-macmillan

@k-macmillan k-macmillan commented Mar 7, 2025

Copy link
Copy Markdown

Description

This was a POC to see if SMS could be sent with send_text_message. This would allow us to use:
PhoneNumber, PhoneNumberId, PhoneNumberArn, SenderId, SenderIdArn, PoolId, or PoolArn.

It uses a different mechanism to send messages and receive updates. This work setup the send message part but did not identify how to receive updates. Without further investigation we cannot implement this.

This did require a bit of effort on Infra to get it functional.

issue N/A

How Has This Been Tested?

Success!
image

Checklist

  • I have assigned myself to this PR
  • PR has an appropriate title: #9999 - What the thing does
  • PR has a detailed description, including links to specific documentation
  • I have added the appropriate labels to the PR.
  • I did not remove any parts of the template, such as checkboxes even if they are not used
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to any documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works. Testing guidelines
  • I have ensured the latest main is merged into my branch and all checks are green prior to review
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • The ticket was moved into the DEV test column when I began testing this change

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant