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

4042 // Enable sorting of columns #1328

Merged
merged 3 commits into from
Jul 17, 2023

Conversation

Dinika
Copy link
Contributor

@Dinika Dinika commented Jul 14, 2023

Fixes #1327

sorting.mp4

Adds basic sorting to all columns.

How has this been tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added necessary unit and integration tests.
  • I have added screenshots (if applicable), in the comment section.

@codecov-commenter
Copy link

codecov-commenter commented Jul 14, 2023

Codecov Report

Merging #1328 (2e6315f) into us-4023/reset-predicate (c1509c5) will decrease coverage by 0.77%.
The diff coverage is 92.30%.

❗ Current head 2e6315f differs from pull request most recent head e5a82a0. Consider uploading reports for the commit e5a82a0 to get more accurate results

@@                     Coverage Diff                     @@
##           us-4023/reset-predicate    #1328      +/-   ##
===========================================================
- Coverage                    47.73%   46.97%   -0.77%     
===========================================================
  Files                          241      237       -4     
  Lines                        10681    10630      -51     
  Branches                      2497     2492       -5     
===========================================================
- Hits                          5099     4993     -106     
- Misses                        5551     5605      +54     
- Partials                        31       32       +1     
Impacted Files Coverage Δ
src/subapps/dataExplorer/PredicateSelector.tsx 96.50% <91.30%> (ø)
src/subapps/dataExplorer/DataExplorer.tsx 100.00% <100.00%> (ø)
src/subapps/dataExplorer/DataExplorerTable.tsx 91.78% <100.00%> (+0.35%) ⬆️

... and 15 files with indirect coverage changes

Comment on lines +118 to +122
sorter: (a, b) => {
return JSON.stringify(a[colName] ?? '').localeCompare(
JSON.stringify(b[colName] ?? '')
);
},
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't know it's just a thought,
I think comparing projects will be more useful only to test the project part not the org part
so instead comparing org/project will only compare project

@Dinika Dinika force-pushed the us-4042/enable-column-sort branch from 2e6315f to de77b08 Compare July 17, 2023 08:47
@Dinika Dinika force-pushed the us-4042/enable-column-sort branch from e5a82a0 to c3e2e91 Compare July 17, 2023 09:12
@Dinika Dinika merged commit de42ee1 into us-4023/reset-predicate Jul 17, 2023
1 check failed
@Dinika Dinika deleted the us-4042/enable-column-sort branch July 17, 2023 09:12
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.

3 participants