Skip to content

Fix conditional copy page#242

Merged
chao-xian merged 2 commits intomainfrom
fix-conditional-copy-page
Apr 2, 2026
Merged

Fix conditional copy page#242
chao-xian merged 2 commits intomainfrom
fix-conditional-copy-page

Conversation

@chao-xian
Copy link
Copy Markdown
Contributor

@chao-xian chao-xian commented Mar 26, 2026

What problem does this pull request solve?

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

Related: alphagov/forms-deploy#2057

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?

@chao-xian chao-xian force-pushed the fix-conditional-copy-page branch from 22e9158 to 781f4c2 Compare March 27, 2026 16:38
@chao-xian chao-xian force-pushed the fix-conditional-copy-page branch from 781f4c2 to 8098433 Compare March 31, 2026 11:25
@chao-xian chao-xian enabled auto-merge March 31, 2026 12:23
Copy link
Copy Markdown
Contributor

@thomasiles thomasiles left a comment

Choose a reason for hiding this comment

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

Worth a go, we'll see if it picks up the value or not?

@chao-xian chao-xian added this pull request to the merge queue Apr 2, 2026
Merged via the queue into main with commit d8690c7 Apr 2, 2026
3 checks passed
@chao-xian chao-xian deleted the fix-conditional-copy-page branch April 2, 2026 09:21
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.

3 participants