-
Notifications
You must be signed in to change notification settings - Fork 128
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
MBMS-73710: Self-Service: Pre-Need: Integration: Service Periods: New Component Design #33504
MBMS-73710: Self-Service: Pre-Need: Integration: Service Periods: New Component Design #33504
Conversation
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…-and-Pre-Integration-Military-History-Removing-a-service-period-sometimes-removes-the-wrong-entry-2
…re-need integration form" This reverts commit 3edd47b.
…re-need integration form" This reverts commit 5a42e4c.
…re-need integration form" This reverts commit 22c7138.
…n for pre-need integration form
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.
src/applications/pre-need-integration/config/pages/servicePeriodsPages.jsx
Outdated
Show resolved
Hide resolved
src/applications/pre-need-integration/config/pages/servicePeriodsPages.jsx
Outdated
Show resolved
Hide resolved
src/applications/pre-need-integration/config/pages/servicePeriodsPages.jsx
Outdated
Show resolved
Hide resolved
src/applications/pre-need-integration/config/pages/servicePeriodsPages.jsx
Outdated
Show resolved
Hide resolved
src/applications/pre-need-integration/config/pages/servicePeriodsPages.jsx
Outdated
Show resolved
Hide resolved
Only change since last approval by Kenny (@ksantiagoBAH) was disabling 1 other failing unit test. |
Refactored page titles to resolve nested ternaries. |
Commenting on the PR so that it will show closed in Jira and stop throwing a warning. |
Jira Link: MBMS-73710
Are you removing, renaming or moving a folder in this PR?
Examples of a TeamSite: https://va.gov/health and https://benefits.va.gov/benefits/. This scenario is also referred to as the "injected" header and footer. You can reach out in the
#sitewide-public-websites
Slack channel for questions.Did you change site-wide styles, platform utilities or other infrastructure?
Summary
As a Veteran and/or family member, I need to be able to input my Service Period(s) so that I can provide the necessary information needed to get a quick determination on possible eligibility.
Testing done
Screenshots
Before:
⚙️TEST DATA ONLY
Screen.Recording.2024-12-24.at.11.32.06.mov
After:
⚙️TEST DATA ONLY
Screen.Recording.2024-12-24.at.11.29.30.mov
What areas of the site does it impact?
Pre-need form 40-10007: Integration
Acceptance criteria
Quality Assurance & Testing
Error Handling
Authentication