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

Rework group selection interface #156

Merged
merged 10 commits into from
Nov 2, 2023

Conversation

brianlove
Copy link
Contributor

@brianlove brianlove commented Oct 24, 2023

Update the group selection interface to be within the 'Name' column's dropdown instead of a separate modal. Update tests accordingly, including adding tests to ensure that filtering functions are working.

Closes #94

@brianlove brianlove requested a review from jmelot October 24, 2023 20:53
@brianlove brianlove self-assigned this Oct 24, 2023
@github-actions
Copy link

github-actions bot commented Oct 24, 2023

No need for rebasing 👍
behind_count is 0
ahead_count is 10

@jmelot jmelot force-pushed the 92-adjust-available-columns branch from a6d8b4a to 2b9372f Compare October 30, 2023 16:30
@brianlove brianlove force-pushed the 92-adjust-available-columns branch 3 times, most recently from 993c151 to 5a863d5 Compare October 30, 2023 18:24
@brianlove brianlove mentioned this pull request Oct 31, 2023
Base automatically changed from 92-adjust-available-columns to version2 October 31, 2023 16:21
Add company groups to the compnay name dropdown.

Note: the old group selection interface is still present for the time
being (and the group selection test fails)
Updated test coverage for groups selection to accomodate the new
interface.  Added tests checking that rows are filtered as expected.
@brianlove brianlove force-pushed the 94-rework-group-selection-interface branch from de09319 to 5556c6f Compare October 31, 2023 19:52
@brianlove brianlove marked this pull request as ready for review October 31, 2023 19:54
Copy link
Member

@jmelot jmelot left a comment

Choose a reason for hiding this comment

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

This looks good to me, modulo the tests. What do you think about opening more issues for adding the footer back in and making the selected groups persistent?

Add and configure an Emotion cache to turn off the "potentially unsafe"
error messages that show up even though they're not applicable.
Copy link

github-actions bot commented Nov 1, 2023

JavaScript Coverage

Summary

Lines Statements Branches Functions
Coverage: 64%
64.98% (284/437) 59.78% (110/184) 64.38% (94/146)
Modified Files • (64%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files64.9859.7864.3864.72 
components60.0562.8356.4859.57 
   ListViewTable.jsx94.8279.7997.6795.06245–246, 251–254, 341, 360, 463, 493
static_data83.877587.582.75 
   table_columns.js86.27587.585.18127–129, 163
util85.9639.2886.3688.46 
   testing.js100100100100 

@brianlove
Copy link
Contributor Author

brianlove commented Nov 1, 2023

Summary footer is tracked in #164 #100. The filter persistence was getting resolved by #155 (implementation of #144), but that's not seeming to work anymore‽‽ (I have no idea why) I'll be investigating it in more detail sometime soon.

@jmelot - Other than those separate issues, the tests are (finally) passing

@brianlove brianlove merged commit 93f566d into version2 Nov 2, 2023
3 checks passed
@brianlove brianlove deleted the 94-rework-group-selection-interface branch November 2, 2023 15:18
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