Skip to content

FDG-11185 MSPD dataset when enter Record Date criteria Pages is going away for a data table.#2068

Merged
ntsim99 merged 10 commits intomasterfrom
FDG-11185
Feb 5, 2026
Merged

FDG-11185 MSPD dataset when enter Record Date criteria Pages is going away for a data table.#2068
ntsim99 merged 10 commits intomasterfrom
FDG-11185

Conversation

@jdach
Copy link
Contributor

@jdach jdach commented Feb 4, 2026

JIRA Ticket:

FDG-11185

The following are ALL required for the PR to be merged:

  • Provided testing instructions in JIRA ticket if applicable
  • Provided mocks or additional information in JIRA ticket if applicable
  • Verified cross-browser compatibility: Chrome, Safari, Firefox, Edge
  • Verified mobile/tablet/desktop/monitor responsiveness
  • Verified that this PR does not create any new accessibility issues (via Axe Chrome)
  • Verify all files are covered by at least 90% test coverage if applicable
  • Check for code quality
    + Watch out for irrelevant changes
    + Take time to understand the logic and flow; be on guard for pitfalls
    + Look at handoff between components (esp. number of props)
    + Watch out for literals (vs. variables) for css specs if applicable

@jdach jdach closed this Feb 4, 2026
@jdach jdach reopened this Feb 4, 2026
@jdach jdach marked this pull request as draft February 4, 2026 18:53
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

Total Line Coverage: 92.21% ✅

@jdach jdach marked this pull request as ready for review February 4, 2026 19:49
@jdach jdach changed the title Fdg 11185 FDG-11185 MSPD dataset when enter Record Date criteria Pages is going away for a data table. Feb 4, 2026
@ntsim99 ntsim99 self-assigned this Feb 4, 2026
Copy link
Contributor

@ntsim99 ntsim99 left a comment

Choose a reason for hiding this comment

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

The bug is gone, and the table is functioning across many of the datasets I've tested. Good work on this one.

@ntsim99 ntsim99 merged commit 8e044ca into master Feb 5, 2026
4 checks passed
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.

2 participants