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

MPDX-7955 - List Header functionality #985

Merged
merged 13 commits into from
Aug 23, 2024
Merged

Conversation

dr-bizz
Copy link
Contributor

@dr-bizz dr-bizz commented Jul 31, 2024

Description

In this PR I add functionality to the ListHeader. Most of it is there already but I also remove some unnecessary functionality.

Added:

  • Fixed refresh issue when searching.
  • Adding Mass action back into the ListHeader
  • Adding checkboxes to the ContactRow on the Flows view.

Removed:

  • Starred filter on ListHeader

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels. (Add the label "On Staging" to get the branch automatically merged into staging.)
  • I have requested a review from another person on the project

@dr-bizz dr-bizz changed the title Mpdx 7955 add functionality MPDX-7955 - List Header functionality Jul 31, 2024
Copy link
Contributor

github-actions bot commented Jul 31, 2024

Bundle sizes [mpdx-react]

Compared against 0426efa

No significant changes found

@dr-bizz dr-bizz requested review from wrandall22 and caleballdrin and removed request for wrandall22 July 31, 2024 19:35
@caleballdrin caleballdrin added the Preview Environment Add this label to create an Amplify Preview label Aug 1, 2024
Copy link
Contributor

github-actions bot commented Aug 1, 2024

Preview branch generated at https://MPDX-7955-add-functionality.d3dytjb8adxkk5.amplifyapp.com

Copy link
Contributor

@caleballdrin caleballdrin left a comment

Choose a reason for hiding this comment

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

Great work on this! Just 2 small changes.

@dr-bizz
Copy link
Contributor Author

dr-bizz commented Aug 13, 2024

Test chuck 4 will be resolved when merged with MPDX-7948-creating-appeal-details-pages-2

@dr-bizz dr-bizz force-pushed the MPDX-7948-creating-appeal-details-pages-2 branch from c8577b5 to 43047d2 Compare August 22, 2024 18:27
Base automatically changed from MPDX-7948-creating-appeal-details-pages-2 to main August 22, 2024 18:37
@dr-bizz dr-bizz force-pushed the MPDX-7955-add-functionality branch from 089ef36 to 199afc8 Compare August 22, 2024 19:19
@dr-bizz dr-bizz requested a review from caleballdrin August 22, 2024 19:23
@dr-bizz dr-bizz requested review from canac and removed request for caleballdrin August 22, 2024 19:33
Copy link
Contributor

@canac canac left a comment

Choose a reason for hiding this comment

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

My suggestions are trivial, so I'm going to go ahead and approve.

});
});

it('should disable select all contacts if no contacts', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a big deal, but some of these tests are unnecessarily async.

@dr-bizz dr-bizz enabled auto-merge (squash) August 23, 2024 13:57
@dr-bizz dr-bizz dismissed caleballdrin’s stale review August 23, 2024 13:57

Caleb Cox approved

@dr-bizz dr-bizz merged commit 6642dc6 into main Aug 23, 2024
17 of 18 checks passed
@dr-bizz dr-bizz deleted the MPDX-7955-add-functionality branch August 23, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Preview Environment Add this label to create an Amplify Preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants