Skip to content

VED-982: Remove Default Value In Apigee Env#1297

Merged
Akol125 merged 1 commit intostaging/VED-16-mns-vacc-event-notificationsfrom
VED-982-fix-changes
Mar 16, 2026
Merged

VED-982: Remove Default Value In Apigee Env#1297
Akol125 merged 1 commit intostaging/VED-16-mns-vacc-event-notificationsfrom
VED-982-fix-changes

Conversation

@Akol125
Copy link
Contributor

@Akol125 Akol125 commented Mar 16, 2026

Summary

  • Routine Change
    Bug fix to remove default value in Apigee environment as it is uses a conditional to check if its value is true or not, a default value would always make it true, which would allow mns_environment conditional to default to int configuration even when it is prod.

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.

@github-actions
Copy link
Contributor

This branch is working on a ticket in the NHS England VED JIRA Project. Here's a handy link to the ticket:

VED-982

@Akol125 Akol125 changed the title remove default apigee env VED-982: Remove default value in Apigee Env Mar 16, 2026
@Akol125 Akol125 changed the title VED-982: Remove default value in Apigee Env VED-982: Remove Default Value In Apigee Env Mar 16, 2026
@sonarqubecloud
Copy link

@Akol125 Akol125 merged commit b49dfe4 into staging/VED-16-mns-vacc-event-notifications Mar 16, 2026
18 checks passed
@Akol125 Akol125 deleted the VED-982-fix-changes branch March 16, 2026 12:04
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