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
fix: adds pagination/infinite scroll to owners select on DashboardList and ChartList #10035
fix: adds pagination/infinite scroll to owners select on DashboardList and ChartList #10035
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10035 +/- ##
==========================================
- Coverage 70.47% 68.86% -1.61%
==========================================
Files 583 584 +1
Lines 31019 31061 +42
Branches 3175 3182 +7
==========================================
- Hits 21861 21391 -470
- Misses 9049 9561 +512
Partials 109 109
Continue to review full report at Codecov.
|
@@ -347,10 +348,6 @@ describe('ListView with new UI filters', () => { | |||
expect(wrapper.find(ListViewFilters)).toHaveLength(1); | |||
}); | |||
|
|||
it('fetched selects if function is provided', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fetching now happens on select open/click. This is really hard to simulate via enzyme.
@@ -198,7 +198,7 @@ const ListView: FunctionComponent<Props> = ({ | |||
{bulkActions.map(action => ( | |||
// @ts-ignore | |||
<MenuItem | |||
key={action.key} | |||
key={action.key || action.name} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when will an action not have a key?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this after getting a react error in tests. I've updated the BulkAction type and made key a required field. Also fixed the test
98819e1
to
d65341f
Compare
SUMMARY
followup from #9989
This PR adds infinite scroll via
react-select-async-paginate
(same author asreact-select
).TEST PLAN
ADDITIONAL INFORMATION