Skip to content

Mod/dev 14599 Tanstack Query for Adv Search Results Table#5195

Open
jagu65 wants to merge 17 commits intoqatfrom
mod/DEV-14599
Open

Mod/dev 14599 Tanstack Query for Adv Search Results Table#5195
jagu65 wants to merge 17 commits intoqatfrom
mod/DEV-14599

Conversation

@jagu65
Copy link
Copy Markdown
Contributor

@jagu65 jagu65 commented Apr 1, 2026

Description:
Refactored ResultsTableContainer to move parameter preparation and data fetching to a custom hook. The data fetching is now utilizing useQuery.

JIRA Ticket:
DEV-14599

Mockup:
n/a

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

Author:

  • Linked to this PR in JIRA ticket
  • Scheduled and completed design review
  • Provided instructions for testing in JIRA ticket and PR 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 extension or Lighthouse report)
  • Added Unit Tests for helper functions, reducers, models and Container/Component Interactivity Expectations if applicable React Testing Library
  • API contract updated if applicable

@nick-torres nick-torres self-assigned this Apr 1, 2026
@nick-torres nick-torres added do not merge pending pr merging of another usaspending-website pr is needed before merging labels Apr 1, 2026
@jagu65 jagu65 removed do not merge pending pr merging of another usaspending-website pr is needed before merging labels Apr 14, 2026
@jagu65 jagu65 changed the title Mod/dev 14599 Mod/dev 14599 Tanstack Query for Adv Search Results Table Apr 16, 2026
@nick-torres
Copy link
Copy Markdown
Contributor

conflicts + 422 error when switching to subawards bc it's attempting to sort on award amount but that field doesn't exist for subawards

@jagu65
Copy link
Copy Markdown
Contributor Author

jagu65 commented Apr 20, 2026

conflicts + 422 error when switching to subawards bc it's attempting to sort on award amount but that field doesn't exist for subawards

Fixed!

@anjenkin anjenkin added do not merge pending sprint start to be merged after code freeze ends, at the start of a new sprint labels Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge pending sprint start to be merged after code freeze ends, at the start of a new sprint

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants