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

APPEALS/29343 - Fix tabPaginationOptions PropType for QueueTable #19319

Merged
merged 5 commits into from
Oct 3, 2023

Conversation

brandondorner
Copy link
Contributor

Resolves: https://jira.devops.va.gov/browse/APPEALS-29343

Description

This is a tech-debt subtask for the tech debt epic https://jira.devops.va.gov/browse/APPEALS-29280.

This resolves a proptype issue for QueueTable by ensuring savedPaginationOptions is an object and not a boolean.
More details can be found in Jira ticket.

Acceptance Criteria

  • Code compiles correctly
  • QueueTable proptype warning is gone
  • The QueueTable pagination options persist

To test the pagination stuff still works we can make sure the url params
(?tab=assigned_person&page=1&sort_by=typeColumn&order=asc are as expected and refreshing the page displays the table/tabs as presented in the url params.

If there is a user with more than one page of cases or if someone can provide a rails command to add cases easily we can test the pagination aspect too.

Jira: https://jira.devops.va.gov/browse/APPEALS-29343

This resolves a proptype issue for QueueTable by ensuring
`savedPaginationOptions` is an object and not a boolean.
@brandondorner brandondorner added Type: Tech-Improvement Team: Saturn 🪐 A development team for the VHA business line labels Aug 30, 2023
@codeclimate
Copy link

codeclimate bot commented Aug 30, 2023

Code Climate has analyzed commit 0aa7db8 and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Contributor

@TylerBroyles TylerBroyles left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@craigrva craigrva merged commit b366dd4 into master Oct 3, 2023
15 checks passed
@craigrva craigrva deleted the dorner/APPEALS-29343 branch October 3, 2023 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team: Saturn 🪐 A development team for the VHA business line Type: Tech-Improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants