Skip to content

Add flag for using admin database as source of truth#1671

Merged
lfdebrux merged 1 commit into
mainfrom
add-use-database-as-truth-flag-in-all-envs
Aug 15, 2025
Merged

Add flag for using admin database as source of truth#1671
lfdebrux merged 1 commit into
mainfrom
add-use-database-as-truth-flag-in-all-envs

Conversation

@SamJamCul
Copy link
Copy Markdown
Contributor

When this flag is enabled in an environment, the forms-admin app will use its own database as the source of information about forms, rather than Forms API. It will still update the API with any form changes

What problem does this pull request solve?

Trello card: https://trello.com/c/tm4TfyDy/2411-add-configuration-flag-for-forms-admin-repositories-source-of-truth

Adds the flag use_database_as_truth to each env for forms admin, as part of this stuff

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, user-research, and/or prod)
  • Check the #govuk-forms-deployment-notifications Slack channel to ensure the apply-forms-terraform-<environment> pipelines have run successfully

When this flag is enabled in an environment, the forms-admin app will
use its own database as the source of information about forms, rather
than Forms API. It will still update the API with any form changes
@SamJamCul SamJamCul force-pushed the add-use-database-as-truth-flag-in-all-envs branch from 0d3a609 to 4121532 Compare August 14, 2025 14:36
@lfdebrux lfdebrux merged commit 2eb7c3b into main Aug 15, 2025
3 checks passed
@lfdebrux lfdebrux deleted the add-use-database-as-truth-flag-in-all-envs branch August 15, 2025 10:05
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