Skip to content
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

Conversation

adam-antonioli
Copy link
Contributor

@adam-antonioli adam-antonioli commented Dec 11, 2024

Jira Link: MBMS-73710

Are you removing, renaming or moving a folder in this PR?

  • No, I'm not changing any folders (skip to TeamSites and delete the rest of this section)
  • Yes, I'm removing, renaming or moving a folder

⚠️ TeamSites ⚠️

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

  • Local testing

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

Screenshot 2024-12-24 at 11 17 46 Screenshot 2024-12-24 at 11 17 58 Screenshot 2024-12-24 at 11 18 09 Screenshot 2024-12-24 at 11 18 20 Screenshot 2024-12-24 at 11 18 35

Quality Assurance & Testing

  • I fixed|updated|added unit tests and integration tests for each feature (if applicable).
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
  • Linting warnings have been addressed
  • Documentation has been updated (link to documentation *if necessary)
  • Screenshot of the developed feature is added
  • Accessibility testing has been performed

Error Handling

  • Browser console contains no warnings or errors.
  • Events are being sent to the appropriate logging solution
  • Feature/bug has a monitor built into Datadog or Grafana (if applicable)

Authentication

  • Did you login to a local build and verify all authenticated routes work as expected with a test user

adam-antonioli and others added 30 commits August 20, 2024 15:12
…-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
@va-vfs-bot va-vfs-bot temporarily deployed to master/MBMS-73710-Self-Service-Military-History-Array-Builder-Design/main December 24, 2024 14:02 Inactive
@va-vfs-bot va-vfs-bot temporarily deployed to master/MBMS-73710-Self-Service-Military-History-Array-Builder-Design/main December 24, 2024 14:17 Inactive
@va-vfs-bot va-vfs-bot temporarily deployed to master/MBMS-73710-Self-Service-Military-History-Array-Builder-Design/main December 24, 2024 14:43 Inactive
@adam-antonioli adam-antonioli marked this pull request as ready for review December 24, 2024 15:16
@adam-antonioli adam-antonioli requested review from a team as code owners December 24, 2024 15:16
Copy link

@va-vfs-bot va-vfs-bot left a 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.

@adam-antonioli
Copy link
Contributor Author

Only change since last approval by Kenny (@ksantiagoBAH) was disabling 1 other failing unit test.

@adam-antonioli adam-antonioli changed the title Mbms 73710 self service military history array builder design MBMS-73710: Self-Service: Pre-Need: Integration: Service Periods: New Component Design Dec 24, 2024
@adam-antonioli
Copy link
Contributor Author

Refactored page titles to resolve nested ternaries.

@va-vfs-bot va-vfs-bot temporarily deployed to master/MBMS-73710-Self-Service-Military-History-Array-Builder-Design/main January 2, 2025 15:21 Inactive
@va-vfs-bot va-vfs-bot temporarily deployed to master/MBMS-73710-Self-Service-Military-History-Array-Builder-Design/main January 2, 2025 15:40 Inactive
@sonntag-adam sonntag-adam self-requested a review January 3, 2025 14:38
@adam-antonioli adam-antonioli merged commit 72cbc41 into main Jan 3, 2025
69 checks passed
@adam-antonioli adam-antonioli deleted the MBMS-73710-Self-Service-Military-History-Array-Builder-Design branch January 3, 2025 16:44
@MichelleDieudonne
Copy link

Commenting on the PR so that it will show closed in Jira and stop throwing a warning.

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.

7 participants