-
Notifications
You must be signed in to change notification settings - Fork 118
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
Eta September 2024 nvn/eea warning callout #6788
Eta September 2024 nvn/eea warning callout #6788
Conversation
Adding callout text.
Moving ETA callout after review of draft content.
Moving September 2024 ETA partial after review.
Adding snippet of new callout to @eta_test
@@ -3,6 +3,7 @@ | |||
<% end %> | |||
|
|||
<% govspeak_for :body do %> | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need this extra line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have removed this one.
The first commit message has a "WIP" can we reword the message? |
@@ -18,7 +18,8 @@ class CheckUkVisaFlowTest < ActiveSupport::TestCase | |||
@b1_b2_country = "syria" | |||
@youth_mobility_scheme_country = "canada" | |||
|
|||
@eta_text = "If you’re travelling on or after 22 February 2024, you’ll need to apply for an electronic travel authorisation (ETA)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was this a redundant text that we just removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No—it was split into two separate pieces of text (one for non-visa nationals, and one for EEA country nationals). See the next two added lines.
Depending on whether the passport holder's country is an EEA country or a non-visa national country, show a different date in the callout box text. Add tests to ensure that when the passport holder's country is neither an EEA country nor a non-visa national country, that no ETA callout text is shown.
No confirmed date for either user group yet. But we know users are looking for assurance, so adding a holding message. Keeping the 2 separate callouts so we can add dates to separate groups when they are ready.
Update the tests for the check UK visa flow to reflect the changes made to the outcome text related to non-visa national and EEA countries and their need for electronic travel authorisation.
0ff6e80
to
62e578f
Compare
I guess this was a WIP commit, but to get rid of it now would involve messing with the entire branch history, which I don't really want to do at this point. |
Content storytime: https://trello.com/c/SwjCds1e/263-3-april-eta-callout-for-2-user-groups-on-check-if-you-need-a-uk-visa
Adding in a callout for non-visa national and EEA user groups that they may need an ETA if coming to the UK after September 2024.
Supersedes #6729, since the preview environment for that PR has expired.
Follow these steps if you are doing a Rails upgrade.