Skip to content

fix(mitxonline): corrected service ids from #5158 - #5262

Merged
Ardiea merged 1 commit into
mainfrom
md/202608_fix_service_ids
Aug 5, 2026
Merged

fix(mitxonline): corrected service ids from #5158#5262
Ardiea merged 1 commit into
mainfrom
md/202608_fix_service_ids

Conversation

@Ardiea

@Ardiea Ardiea commented Aug 5, 2026

Copy link
Copy Markdown
Member

Description (What does it do?)

Corrects service ids from #5158 that were entered wrong. Previously was the learn-ai service ids, needs to be the mit-learn service ids.

Copilot AI lite review requested due to automatic review settings August 5, 2026 15:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Corrects the fastly.mit_learn_service_id SOPS-encrypted values for mitxonline in QA and Production, fixing the mis-entered service IDs introduced in #5158 (learn-ai IDs instead of MIT Learn IDs).

Changes:

  • Updated the SOPS-encrypted fastly.mit_learn_service_id value in secrets.qa.yaml.
  • Updated the SOPS-encrypted fastly.mit_learn_service_id value in secrets.production.yaml.
  • Regenerated SOPS metadata blocks as part of the re-encryption (e.g., ordering/empty KMS provider arrays, lastmodified, mac).

Reviewed changes

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

File Description
src/bridge/secrets/mitxonline/secrets.qa.yaml Updates QA’s encrypted fastly.mit_learn_service_id and refreshes SOPS metadata.
src/bridge/secrets/mitxonline/secrets.production.yaml Updates Production’s encrypted fastly.mit_learn_service_id and refreshes SOPS metadata.

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

@Ardiea
Ardiea merged commit dd15a62 into main Aug 5, 2026
7 checks passed
@Ardiea
Ardiea deleted the md/202608_fix_service_ids branch August 5, 2026 15:39
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