Skip to content

Add new multiple routes to large admin post list#2116

Merged
thomasiles merged 1 commit into
mainfrom
add-multiple-routes-to-large-admin-posts
May 12, 2026
Merged

Add new multiple routes to large admin post list#2116
thomasiles merged 1 commit into
mainfrom
add-multiple-routes-to-large-admin-posts

Conversation

@thomasiles
Copy link
Copy Markdown
Contributor

Add new multiple routes to large admin post list

Trello card: https://trello.com/c/kdSpmSY2/3094-allow-larger-post-request-to-multiple-routes-page

The new page for multiple routes requires submitting a larger POST body.

The new page scales based on the number of pages and the number selection options in the form. The full design isn't finalised yet, and the feature is currently behind a feature flag.

Allowing the page to use larger POST bodies now will allow us to test the feature with larger forms.

I followed the pattern for the Welsh page. I've tested this on dev and it worked!

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

Reminders

If you've made changes to the deployer role (files in modules/deployer-access):

  • Remember to run make <environment> forms/account apply on the relevant environments (dev, staging and/or prod)
  • Check the #govuk-forms-deployment-notifications Slack channel to ensure the apply-forms-terraform-<environment> pipelines have run successfully

The new page for multiple routes requires submitting a larger POST body.

The new page scales based on the number of pages and the number
selection options in the form. The full design isn't finalized yet, and
the feature is currently behind a feature flag.

Allowing the page to use larger POST bodies now will allow us to test
the feature with larger forms.
@thomasiles thomasiles added this pull request to the merge queue May 12, 2026
Merged via the queue into main with commit 141b307 May 12, 2026
19 checks passed
@thomasiles thomasiles deleted the add-multiple-routes-to-large-admin-posts branch May 12, 2026 08:56
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