Skip to content

Remove forms-product-page support URL from forms-admin envvars - #2323

Merged
lfdebrux merged 2 commits into
mainfrom
ldeb-add-forms-admin-variable-product-page-base-url
Oct 19, 2026
Merged

lfdebrux merged 2 commits into
mainfrom
ldeb-add-forms-admin-variable-product-page-base-url

Conversation

@lfdebrux

Copy link
Copy Markdown
Contributor

What problem does this pull request solve?

forms-admin no longer needs the forms-product-page support URL as an environment variable (see PR govuk-forms/forms-admin#3207), so we can remove it from our Terraform.

Also, while working on these changes I realised that in some places we are using forms.service.gov.uk in some places for the product pages base URL, but that is just a redirect to www.forms.service.gov.uk, so we can reduce requests clients need to make by making sure we include the www. subdomain when writing URLs for forms-product-page.

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 forms-product-page app is hosted at `www.forms.service.gov.uk`.
However, in places where was pass the URL for that app to other
apps/services, we've been using `forms.service.gov.uk`. This does work
because there is a 301 permanent redirect from `forms.service.gov.uk` to
`www.`, but it might be preferable to avoid that redirect when it's not
necessary.

This commit updates any places where we have written the product pages
base URL to make sure that the `www.` subdomain part is included.
forms-admin no longer needs the forms-product-page support URL as an
environment variable (see PR govuk-forms/forms-admin#3207), so we can
remove it from our Terraform.
@lfdebrux
lfdebrux added this pull request to the merge queue Oct 19, 2026
Merged via the queue into main with commit 5cd0b20 Oct 19, 2026
20 checks passed
@lfdebrux
lfdebrux deleted the ldeb-add-forms-admin-variable-product-page-base-url branch October 19, 2026 05:48
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