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

Update countries to require an ETA (EU/EEA group) #7139

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PeterHattyar
Copy link

Trello card

Countries that were in the COUNTRY_GROUP_ETA_ROLLOUT_GROUP_2_EU_EEA group should now be requiring an electronic travel authorisation (ETA). All countries have been moved.

This follows on the rollout of ETA requirements, now for EU/EEA countries, continuing the work previously from
Trello card
PR #7031

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

… group should now be requiring an electronic travel authorisation (ETA). All countries have been moved.
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7139 March 26, 2025 16:07 Inactive
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7139 March 26, 2025 21:31 Inactive
- Remove now deprecated tests referencing the separate group.
- Remove containers for the separate group.
- Adjust tests broken by the change to correct outcome for nodes now with ETA specific outcome for EEA countries.
- eea_country container still in place as the set COUNTRY_GROUP_EEA has Ireland, which is not part of the EU ETA list.
- non_visa_national_country container value now set to country that is non-visa, but not on the EU ETA list.
@baisa baisa force-pushed the move-eta-rollout-group-2-eu-eea-countries-to-eta-list branch from 26904ee to 7bbf962 Compare March 27, 2025 11:40
@baisa baisa added the DO NOT MERGE Needs further approval before release label Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Needs further approval before release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants