Skip to content

Add ability to enable receiving weekly submission batches#2652

Merged
stephencdaly merged 6 commits intomainfrom
add-ui-for-enabling-weekly-submissions
Mar 26, 2026
Merged

Add ability to enable receiving weekly submission batches#2652
stephencdaly merged 6 commits intomainfrom
add-ui-for-enabling-weekly-submissions

Conversation

@stephencdaly
Copy link
Contributor

What problem does this pull request solve?

Trello card: https://trello.com/c/6ebxtzqO

Adds a weekly_submission_emails_enabled feature flag that these changes are behind.

If the feature flag is enabled:

  • Display a second checkbox on the page to enable weekly batch submissions and different page content
  • Display different success messages when daily/weekly batches are enabled/disabled
  • Use different wording for the task in the task list
  • Display different content on the live form 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?
  • Do the end to end tests need updating before these changes will pass?
  • Has all relevant documentation been updated?

When enabled, we will show the option to get weekly submission
batches.
@stephencdaly stephencdaly force-pushed the add-ui-for-enabling-weekly-submissions branch 2 times, most recently from a102afe to 5de0a95 Compare March 25, 2026 10:23
@stephencdaly stephencdaly marked this pull request as ready for review March 25, 2026 10:29
@stephencdaly stephencdaly force-pushed the add-ui-for-enabling-weekly-submissions branch 2 times, most recently from 0e72c83 to 3f8c584 Compare March 25, 2026 11:19
@stephencdaly stephencdaly force-pushed the add-ui-for-enabling-weekly-submissions branch from 3f8c584 to 4858da8 Compare March 25, 2026 12:12
If the weekly_submission_emails_enabled feature flag is enabled, show
an additional checkbox to enable sending weekly batches of
submissions.

If the feature flag is disabled, continue showing a single checkbox.

Use different content on the page depending on whether the feature
flag is enabled or not.
If the weekly_submission_emails_enabled feature flag is enabled,
display a success message when the batch submissions settings are
changed depending on which settings were enabled or disabled.

Continue to show the old success message if the feature flag is
disabled.
Return "completed" status for the task to enable batch submissions
if either daily or weekly batches are enabled.
If the weekly_submission_emails_enabled feature flag is enabled,
display different content for teh batch submissions task list item.
Update the content displayed on the live form page when the
weekly_submission_emails_enabled feature flay is enabled to have
a different title and a paragraph describing which combination of
daily and weekly batches are enabled.
@stephencdaly stephencdaly force-pushed the add-ui-for-enabling-weekly-submissions branch from 4858da8 to 07c108e Compare March 25, 2026 15:08
@stephencdaly stephencdaly requested a review from hannahkc March 25, 2026 15:09
@sonarqubecloud
Copy link

@github-actions
Copy link

🎉 A review copy of this PR has been deployed! You can reach it at: https://pr-2652.admin.review.forms.service.gov.uk/

It may take 5 minutes or so for the application to be fully deployed and working. If it still isn't ready
after 5 minutes, there may be something wrong with the ECS task. You will need to go to the integration AWS account
to debug, or otherwise ask an infrastructure person.

For the sign in details and more information, see the review apps wiki page.

Copy link
Contributor

@hannahkc hannahkc left a comment

Choose a reason for hiding this comment

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

All the content looks perfecto! Thanks!

@stephencdaly stephencdaly added this pull request to the merge queue Mar 26, 2026
Merged via the queue into main with commit 8454395 Mar 26, 2026
6 checks passed
@stephencdaly stephencdaly deleted the add-ui-for-enabling-weekly-submissions branch March 26, 2026 11:24
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