Skip to content

LG-7789: Update the International Phone Number Support help page#7154

Merged
jmdembe merged 7 commits intomainfrom
LG-7789-update-international-number
Oct 19, 2022
Merged

LG-7789: Update the International Phone Number Support help page#7154
jmdembe merged 7 commits intomainfrom
LG-7789-update-international-number

Conversation

@jmdembe
Copy link
Contributor

@jmdembe jmdembe commented Oct 17, 2022

🎫 Ticket

LG-7789

🛠 Summary of changes

This ticket adds/updates the following information to the international countries list:

  • Add American Samoa to pinpoint_overrides.yml
  • Edit Northern Mariana Islands to show that SMS is not supported

Once changed, the table for "International phone number support" will reflect the API change

📜 Testing Plan

?

👀 Screenshots

If relevant, include a screenshot or screen capture of the changes.

Before
image

After
image
image

🚀 Notes for Deployment

N/A

@jmdembe jmdembe requested a review from a team October 17, 2022 15:53
Copy link
Contributor Author

@jmdembe jmdembe left a comment

Choose a reason for hiding this comment

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

Quick question: I ran make update_pinpoint_supported_countries and it updated information for Japan--it changes to show that voice is no longer supported. Should we add the change for a quick win?

@jmdembe jmdembe requested a review from mitchellhenke October 17, 2022 15:57
changelog: Internal, API improvements, add and update information for American Samoa and Northern Mariana Islands
@jmdembe jmdembe force-pushed the LG-7789-update-international-number branch from c8e9b64 to c6b529c Compare October 17, 2022 16:26
@mitchellhenke
Copy link
Contributor

Quick question: I ran make update_pinpoint_supported_countries and it updated information for Japan--it changes to show that voice is no longer supported. Should we add the change for a quick win?

Yep, please do! I think we already disabled it on our side, so it shouldn't end up being different to end users.

@jmdembe
Copy link
Contributor Author

jmdembe commented Oct 17, 2022

Quick question: I ran make update_pinpoint_supported_countries and it updated information for Japan--it changes to show that voice is no longer supported. Should we add the change for a quick win?

Yep, please do! I think we already disabled it on our side, so it shouldn't end up being different to end users.

Sounds good. I'll make the update

@jmdembe jmdembe force-pushed the LG-7789-update-international-number branch from 0727a65 to d7d3685 Compare October 17, 2022 19:36
@jmdembe jmdembe merged commit bea75e9 into main Oct 19, 2022
@jmdembe jmdembe deleted the LG-7789-update-international-number branch October 19, 2022 17:10
mdiarra3 pushed a commit that referenced this pull request Oct 19, 2022
* Add info for American Samoa, edit info for Northern Mariana Islands

changelog: Internal, API improvements, add and update information for American Samoa and Northern Mariana Islands

* fix lint error

* update information for Japan

* revert `supports_sms` value for MP and AS

* revert based on discussion

* Revert "revert based on discussion"

This reverts commit 2e696f8.
mdiarra3 added a commit that referenced this pull request Oct 19, 2022
…) (#7175)

* Add info for American Samoa, edit info for Northern Mariana Islands

changelog: Internal, API improvements, add and update information for American Samoa and Northern Mariana Islands

* fix lint error

* update information for Japan

* revert `supports_sms` value for MP and AS

* revert based on discussion

* Revert "revert based on discussion"

This reverts commit 2e696f8.

Co-authored-by: Jessica Dembe <jessica.dembe@gsa.gov>
This was referenced Oct 20, 2022
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.

2 participants