Skip to content

Chore: [AEA-6580] - enable notifications for Knights, Day Lewis sites#2989

Merged
tstephen-nhs merged 2 commits intomainfrom
aea-6580-knights-day-lewis
Apr 23, 2026
Merged

Chore: [AEA-6580] - enable notifications for Knights, Day Lewis sites#2989
tstephen-nhs merged 2 commits intomainfrom
aea-6580-knights-day-lewis

Conversation

@tstephen-nhs
Copy link
Copy Markdown
Contributor

Summary

  • 🤖 Operational or Infrastructure Change

Details

  • enable notifications for new sites

@tstephen-nhs tstephen-nhs marked this pull request as ready for review April 23, 2026 09:28
Copilot AI review requested due to automatic review settings April 23, 2026 09:28
Copy link
Copy Markdown
Contributor

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 updates environment configuration for site ODS-code filtering used by the notification pipeline, specifically the blocked ODS code lists for INT and PROD.

Changes:

  • Added 33 new site ODS codes to the PROD blocked list.
  • Added the same 33 new site ODS codes to the INT blocked list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/config/blocked_ods_codes_prod.txt Appends additional ODS codes to the PROD “blocked notifications” list.
.github/config/blocked_ods_codes_int.txt Appends additional ODS codes to the INT “blocked notifications” list.

Comment thread .github/config/blocked_ods_codes_prod.txt
Comment thread .github/config/blocked_ods_codes_prod.txt
Comment thread .github/config/blocked_ods_codes_int.txt
@sonarqubecloud
Copy link
Copy Markdown

@tstephen-nhs tstephen-nhs merged commit c1ab9c1 into main Apr 23, 2026
21 checks passed
@tstephen-nhs tstephen-nhs deleted the aea-6580-knights-day-lewis branch April 23, 2026 09:50
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