Skip to content

Comments

Add exception to sender_id requirement for Australia (AU)#8564

Merged
soniaconnolly merged 4 commits intomainfrom
pinpoint-error
Jun 8, 2023
Merged

Add exception to sender_id requirement for Australia (AU)#8564
soniaconnolly merged 4 commits intomainfrom
pinpoint-error

Conversation

@soniaconnolly
Copy link
Contributor

AWS updated their Supported countries and regions (SMS channel) to say that Australia needs sender ID, but we're not seeing that, so we are adding it to our list of SENDER_ID_COUNTRIES.
https://docs.aws.amazon.com/pinpoint/latest/userguide/channels-sms-countries.html

soniaconnolly and others added 4 commits June 8, 2023 16:03
AWS updated their Supported countries and regions (SMS channel) to say that Australia
needs sender ID, but we're not seeing that, so we are adding it to our list of
SENDER_ID_COUNTRIES.
https://docs.aws.amazon.com/pinpoint/latest/userguide/channels-sms-countries.html

Co-authored-by: Gina Yamada <gina.yamada@gsa.gov>
Co-authored-by: Matt Hinz <matt.hinz@gsa.gov>
We don't want to try to send a sender_id to a country where it is not registered.

Co-authored-by: Zach Margolis <zach.margolis@gsa.gov>
Co-authored-by: Gina Yamada <gina.yamada@gsa.gov>
changelog: Internal, International SMS configuration, Update Australia to not require a sender_id
@soniaconnolly soniaconnolly changed the title Add Australia (AU) to SENDER_ID_COUNTRIES Add exception to sender_id requirement for Australia (AU) Jun 8, 2023
@gina-yamada
Copy link
Contributor

approved!

@matthinz
Copy link
Contributor

matthinz commented Jun 8, 2023

Nice, this is much better than just adding AU to the list and hoping for the best!

@soniaconnolly soniaconnolly merged commit 7dc4cda into main Jun 8, 2023
@soniaconnolly soniaconnolly deleted the pinpoint-error branch June 8, 2023 23:33
@jmhooper jmhooper mentioned this pull request Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants