Skip to content

Update form details link#270

Merged
DavidBiddle merged 1 commit into
mainfrom
update-form-details-link
May 15, 2026
Merged

Update form details link#270
DavidBiddle merged 1 commit into
mainfrom
update-form-details-link

Conversation

@DavidBiddle
Copy link
Copy Markdown
Contributor

We previously updated this link text in forms-admin, which broke the tests.

What problem does this pull request solve?

Trello card: https://trello.com/c/VUw58sEk/2891-add-confirmation-pages-for-english-welsh-when-a-form-version-has-been-made-live

Updates some link text to match a change in admin which broke the tests.

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?
  • Has all relevant documentation been updated?

We previously updated this link text in forms-admin, which broke the tests.
Copy link
Copy Markdown
Contributor

@whi-tw whi-tw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DavidBiddle DavidBiddle added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit b63cf13 May 15, 2026
3 checks passed
@DavidBiddle DavidBiddle deleted the update-form-details-link branch May 15, 2026 09:34
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