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

Fix date issues, mobile and desktop #33753

Merged
merged 2 commits into from
Jan 2, 2025

Conversation

amponce
Copy link
Contributor

@amponce amponce commented Dec 23, 2024

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)

Summary

  • We originally ran into trouble storing partial date strings like “YYYY-MM-XX,” which led to validation failures and off-by-one-year issues on some devices. To fix this, we switched to consistently storing “YYYY-MM” and converting to a valid “YYYY-MM-01” format for validation, removing the time zone confusion. We also updated our helper function (monthYearFormatter) to gracefully handle legacy formats, ensuring a consistent display of “MM/yyyy” across the fsr

Related issue(s)

Testing done

  • Manual verification, visual verifcaiton of final pdf submissions in both desktop and mobile, see attached

Screenshots

Note: This field is mandatory for UI changes (non-component work should NOT have screenshots).

Mobile
image

What areas of the site does it impact?

(Describe what parts of the site are impacted if code touched other areas)

Acceptance criteria

Quality Assurance & Testing

  • I fixed | updated | added cypress 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
  • Screenshot of the developed feature is added
  • Accessibility testing has been performed

Error Handling

  • Browser console contains no warnings or errors.

@amponce amponce self-assigned this Dec 23, 2024
@amponce amponce requested review from a team as code owners December 23, 2024 21:27
@amponce
Copy link
Contributor Author

amponce commented Dec 23, 2024

Test form submissions show date is working as expected

VA Form 5655 - Submitted (11).pdf
VA Form 5655 - Submitted (12).pdf

OptionSelect
OptionSelect previously approved these changes Dec 23, 2024
Copy link
Contributor

@OptionSelect OptionSelect left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@amponce amponce force-pushed the bugfix/99274/fix-mobile-employment-date branch from cbc305a to 5e72ec3 Compare December 23, 2024 21:47
@va-vfs-bot va-vfs-bot temporarily deployed to master/bugfix/99274/fix-mobile-employment-date/main December 23, 2024 21:58 Inactive
@amponce amponce force-pushed the bugfix/99274/fix-mobile-employment-date branch from 5e72ec3 to 037ae32 Compare January 2, 2025 17:28
@va-vfs-bot va-vfs-bot temporarily deployed to master/bugfix/99274/fix-mobile-employment-date/main January 2, 2025 18:07 Inactive
@amponce amponce force-pushed the bugfix/99274/fix-mobile-employment-date branch from 3d19f5d to 9798170 Compare January 2, 2025 18:54
@va-vfs-bot va-vfs-bot temporarily deployed to master/bugfix/99274/fix-mobile-employment-date/main January 2, 2025 19:31 Inactive
Copy link
Contributor

@Dr-Pongo Dr-Pongo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@amponce amponce merged commit 9aa99fa into main Jan 2, 2025
80 checks passed
@amponce amponce deleted the bugfix/99274/fix-mobile-employment-date branch January 2, 2025 19:42
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.

4 participants