-
Notifications
You must be signed in to change notification settings - Fork 120
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
Replace standard visitor with short-term study (Content and Link) For Taiwan #2398
Conversation
0b240c9
to
fbac5c6
Compare
This doesn't need a factcheck with the department, and the preview looks good to me. |
@keithiopia Thanks for letting us know will move on with code review.. |
Minor: I think the commit note subject line for the first commit should mention "Taiwan", because as far as I can see that's the only outcome that is changing. It should be fine to amend the commit note for that commit - @keithiopia will remain the "author" of the commit and you will be the "committer". |
Minor: I think the PR title should mention "Taiwan" and it could probably be less detailed e.g. "Content changes for Taiwan outcome in Check UK Visa". This seems quite important, because the PR description doesn't really mention Taiwan either. |
Very minor: The browser developer tools in the screenshots are a bit distracting. |
Changed visa needed from standard visitor to short-term study. Request was in https://govuk.zendesk.com/agent/tickets/1287220
Other than my minor comments, this looks good to me. 👍 |
fbac5c6
to
8ae29b4
Compare
Zenddesk Story: https://govuk.zendesk.com/agent/tickets/1287220
Trello Story: https://trello.com/c/2NNNCDPL/67-change-study-visa-link
Factchecks
https://smart-answers-pr-2398.herokuapp.com/check-uk-visa/y/taiwan/school
https://smart-answers-pr-2398.herokuapp.com/check-uk-visa/y/taiwan/schootourism
https://smart-answers-pr-2398.herokuapp.com/check-uk-visa/y/taiwan/medical
https://smart-answers-pr-2398.herokuapp.com/check-uk-visa/y/taiwan/study/six_months_or_less
Expected changes
Before
After