Skip to content

Store Welsh from document on submission#2111

Merged
stephencdaly merged 2 commits into
mainfrom
store-welsh-from-document-on-submission
May 20, 2026
Merged

Store Welsh from document on submission#2111
stephencdaly merged 2 commits into
mainfrom
store-welsh-from-document-on-submission

Conversation

@stephencdaly
Copy link
Copy Markdown
Contributor

@stephencdaly stephencdaly commented May 20, 2026

What problem does this pull request solve?

Trello card: https://trello.com/c/hCWhyMOm

In order to send confirmation emails with a copy of the user's answers in Welsh, we need to store the Welsh version of the form document at the time of submission in order to get the Welsh translations for the questions that exist in the form at that moment in time.

This is necessary as we send confirmation emails asynchronously using ActiveJob, so cannot retrieve the document from forms-admin at the time of sending as it may have changed since the submission.

Things to consider when reviewing

  • Ensure that you consider the wider context.
  • Does it work when run on your machine?
  • Is it clear what the code is doing?
  • Do the commit messages explain why the changes were made?
  • Are there all the unit tests needed?
  • Do the end to end tests need updating before these changes will pass?
  • Has all relevant documentation been updated?

In order to send confirmation emails with a copy of the user's answers
in Welsh, we need to store the Welsh version of the form document at
the time of submission in order to get the Welsh translations for the
questions that exist in the form at that moment in time. This is
necessary as we send confirmation emails asynchronously using
ActiveJob, so cannot retrieve the document from forms-admin at the
time of sending as it may have changed since the submission.
If Welsh has been used when completing the form, store the Welsh
form document on the submission. If we've already loaded the Welsh
form document, we can use that - otherwise retrieve it from
forms-admin.
@github-actions
Copy link
Copy Markdown
Contributor

🎉 A review copy of this PR has been deployed! You can reach it at: https://pr-2111.submit.review.forms.service.gov.uk/

It may take 5 minutes or so for the application to be fully deployed and working. If it still isn't ready
after 5 minutes, there may be something wrong with the ECS task. You will need to go to the integration AWS account
to debug, or otherwise ask an infrastructure person.

For the sign in details and more information, see the review apps wiki page.

@stephencdaly stephencdaly changed the title Store welsh from document on submission Store Welsh from document on submission May 20, 2026
@stephencdaly stephencdaly marked this pull request as ready for review May 20, 2026 10:46
@stephencdaly stephencdaly added this pull request to the merge queue May 20, 2026
Merged via the queue into main with commit 214a820 May 20, 2026
6 checks passed
@stephencdaly stephencdaly deleted the store-welsh-from-document-on-submission branch May 20, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants