-
Notifications
You must be signed in to change notification settings - Fork 69
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
VACMS-19227: Adds Philippines as a US state for facility purposes #20068
base: main
Are you sure you want to change the base?
Conversation
Checking composer.lock changes... |
Checking composer.lock changes... |
@edmund-dunn @dsasser I have a change in this PR that replaces the default states in the US with a list that includes the Philippines, as we have facilities in the Philippines that are mapped to US addresses by way of the Lighthouse API: Do you think it makes more sense to keep it in the |
5022f22
to
7578442
Compare
Checking composer.lock changes... |
Based on the conversation on Slack, I chose to create a new module and add the code to that: https://dsva.slack.com/archives/CT4GZBM8F/p1733844761259909 |
Checking composer.lock changes... |
Checking composer.lock changes... |
10553e8
to
72df54e
Compare
Checking composer.lock changes... |
Checking composer.lock changes... |
Tugboat has finished building the preview for this pull request! Links:
Dashboard: |
Cypress Accessibility Violations
|
Description
Relates to #19227
Testing done
Manually
Cypress (regression)
Screenshots
QA steps
Set up QA Content Publisher user
Add Philippines as a state to facilities
Create an event
Set the location for facilities
Create a news release
Edit a pre-existing news release
Regression tests
Select Team for PR review
CMS Team
Public websites
Facilities
User support
Accelerated Publishing