Skip to content

Add Notify API key to forms runner queue worker#1998

Merged
theseanything merged 1 commit intomainfrom
theseanything/add-notify-key-to-runner-queue-workers
Feb 24, 2026
Merged

Add Notify API key to forms runner queue worker#1998
theseanything merged 1 commit intomainfrom
theseanything/add-notify-key-to-runner-queue-workers

Conversation

@theseanything
Copy link
Contributor

This is allows us to send the confirmation email as an async job.

Forgot to do this before I merged this change alphagov/forms-runner#1896.

Copilot AI review requested due to automatic review settings February 24, 2026 14:51
This is allows us to send the confirmation email as an async job.
@theseanything theseanything force-pushed the theseanything/add-notify-key-to-runner-queue-workers branch from a495c8b to 223ed79 Compare February 24, 2026 14:53
@theseanything theseanything requested review from a team and stephencdaly February 24, 2026 14:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the GOV.UK Notify API key configuration to the forms-runner queue worker, enabling it to send confirmation emails as async jobs. This addresses an oversight from PR #1896 where the async email functionality was added but the required API key configuration for the queue worker was not included.

Changes:

  • Added SETTINGS__GOVUK_NOTIFY__API_KEY secret to the queue worker container definition

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@theseanything theseanything added this pull request to the merge queue Feb 24, 2026
Merged via the queue into main with commit 759e0a8 Feb 24, 2026
17 checks passed
@theseanything theseanything deleted the theseanything/add-notify-key-to-runner-queue-workers branch February 24, 2026 15:02
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