Skip to content
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

Fix PhoneNumberBanned producing status code 500 #4570

Merged
merged 3 commits into from
Jun 25, 2024

Conversation

mderynck
Copy link
Contributor

@mderynck mderynck commented Jun 20, 2024

What this PR does

Handle PhoneNumberBanned as status code 403 instead of 500

Which issue(s) this PR closes

Closes [issue link here]

Checklist

  • Unit, integration, and e2e (if applicable) tests updated
  • Documentation added (or pr:no public docs PR label added if not required)
  • Added the relevant release notes label (see labels prefixed w/ release:). These labels dictate how your PR will
    show up in the autogenerated release notes.

@mderynck mderynck added pr:no public docs Added to a PR that does not require public documentation updates release:patch PR will be added to "Other Changes" section of release notes labels Jun 20, 2024
@mderynck mderynck requested a review from a team June 20, 2024 22:10
@joeyorlando joeyorlando added this pull request to the merge queue Jun 25, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 25, 2024
@joeyorlando joeyorlando added this pull request to the merge queue Jun 25, 2024
Merged via the queue into dev with commit 63663b2 Jun 25, 2024
21 checks passed
@joeyorlando joeyorlando deleted the mderynck/phone-number-banned-exception branch June 25, 2024 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:no public docs Added to a PR that does not require public documentation updates release:patch PR will be added to "Other Changes" section of release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants