Skip to content

Added required as false to mns vlidation flag in e2e pipeline#1377

Merged
avshetty1980 merged 2 commits intomasterfrom
bug/VED-0000-changed-mns-validation-flag-to-boolean
Apr 2, 2026
Merged

Added required as false to mns vlidation flag in e2e pipeline#1377
avshetty1980 merged 2 commits intomasterfrom
bug/VED-0000-changed-mns-validation-flag-to-boolean

Conversation

@avshetty1980
Copy link
Copy Markdown
Contributor

@avshetty1980 avshetty1980 commented Apr 2, 2026

Summary

the mns validation flag int he e2e automation test pipeline is a string but needs required as false - there is already a check for this and there is a default value set to true also.

@avshetty1980 avshetty1980 marked this pull request as ready for review April 2, 2026 12:18
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

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

VED-0000

Thomas-Boyle
Thomas-Boyle previously approved these changes Apr 2, 2026
@avshetty1980 avshetty1980 changed the title converted string to boolean in default mns vlidation flag Added required as false to mns vlidation flag in e2e pipeline Apr 2, 2026
@Thomas-Boyle Thomas-Boyle added the bugfix This PR contains a fix for existing functionality. label Apr 2, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 2, 2026

@avshetty1980 avshetty1980 merged commit 5380a6b into master Apr 2, 2026
18 checks passed
@avshetty1980 avshetty1980 deleted the bug/VED-0000-changed-mns-validation-flag-to-boolean branch April 2, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This PR contains a fix for existing functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants