-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[VACMS-19406]Correct Manila Profile Page Leadership Links #2399
base: main
Are you sure you want to change the base?
[VACMS-19406]Correct Manila Profile Page Leadership Links #2399
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ESLint is disabled
vets-website
uses ESLint to help enforce code quality. In most situations we would like ESLint to remain enabled.
What you can do
See if the code can be refactored to avoid disabling ESLint, or wait for a VSP review.
@@ -0,0 +1,70 @@ | |||
/* eslint-disable no-param-reassign */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ESLint disabled here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disabled as the page does need to be reassigned in this build adjustment specific to Manila pages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ESLint is disabled
vets-website
uses ESLint to help enforce code quality. In most situations we would like ESLint to remain enabled.
What you can do
See if the code can be refactored to avoid disabling ESLint, or wait for a VSP review.
@@ -0,0 +1,88 @@ | |||
/* eslint-disable @department-of-veterans-affairs/axe-check-required */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ESLint disabled here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disabled as there is no direct FE components (labels, titles, wording) being affected, with no way for this to do more than adjust the path of a URL on the page.
a8d732b
to
e097ed2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Follows similar idea to process from Lovell with a few less parts.
Summary
Related issue(s)
Testing done
What areas of the site does it impact?
Pages in the Manila VA Clinic
Acceptance criteria
Leadership link in left sidenav should correctly point to the live / published Leadership list for Manila
Existing sidenav Leadership link behavior for Lovell does not regress e.g. https://www.va.gov/lovell-federal-health-care-va/staff-profiles/robert-buckley/
As a VA beneficiary, left sidenav Leadership link points to https://www.va.gov/lovell-federal-health-care-va/about-us/leadership
[x]As a Tricare beneficiary, left sidenav link points to https://www.va.gov/lovell-federal-health-care-tricare/about-us/leadership
Quality Assurance & Testing
Error Handling
Authentication
Requested Feedback
(OPTIONAL) What should the reviewers know in addition to the above. Is there anything specific you wish the reviewer to assist with. Do you have any concerns with this PR, why?