Skip to content

VED-982: MNS Notification Complete#1309

Merged
Akol125 merged 22 commits intomasterfrom
VED-982-MNS-Notification-complete
Mar 19, 2026
Merged

VED-982: MNS Notification Complete#1309
Akol125 merged 22 commits intomasterfrom
VED-982-MNS-Notification-complete

Conversation

@Akol125
Copy link
Contributor

@Akol125 Akol125 commented Mar 17, 2026

Summary

  • Routine Change
  • 🤖 Operational or Infrastructure Change
  • ✨ New Feature
    Merging the full MNS Publication work from staging into Master

Add any other relevant notes or explanations here. Remove this line if you have nothing to add.

Reviews Required

  • Dev
  • Test
  • Tech Author
  • Product Owner

Review Checklist

ℹ️ This section is to be filled in by the reviewer.

  • I have reviewed the changes in this PR and they fill all of the acceptance criteria of the ticket.
  • If there were infrastructure, operational, or build changes, I have made sure there is sufficient evidence that the changes will work.
  • If there were changes that are outside of the regular release processes e.g. account infrastructure to setup, manual setup for external API integrations, secrets to set, then I have checked that the developer has flagged this to the Tech Lead as release steps.
  • I have checked that no Personal Identifiable Data (PID) is logged as part of the changes.

@Akol125 Akol125 changed the title Ved 982 mns notification complete VED-982: Mns Notification Complete Mar 17, 2026
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 17, 2026 10:14 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 17, 2026 10:14 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 17, 2026 10:15 — with GitHub Actions Inactive
avshetty1980
avshetty1980 previously approved these changes Mar 17, 2026
Copy link
Contributor

@avshetty1980 avshetty1980 left a comment

Choose a reason for hiding this comment

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

LGTM

@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 18, 2026 20:48 — with GitHub Actions Inactive
@Akol125 Akol125 had a problem deploying to internal-dev-sandbox March 18, 2026 20:49 — with GitHub Actions Failure
@Akol125 Akol125 had a problem deploying to internal-dev-sandbox March 18, 2026 20:49 — with GitHub Actions Failure
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 18, 2026 21:15 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 18, 2026 21:16 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 18, 2026 23:41 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 18, 2026 23:42 — with GitHub Actions Inactive
Copy link
Contributor

@avshetty1980 avshetty1980 left a comment

Choose a reason for hiding this comment

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

LGTM. We can raise a follow-up ticket for having the same KMS key for the shared outbound queue between green/blue envs.

@sonarqubecloud
Copy link

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.

6 participants