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: remove -1,-1 from owners request in charts, dashboards list and propertiesModal #9989
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9989 +/- ##
=======================================
Coverage 71.38% 71.38%
=======================================
Files 585 585
Lines 30987 30994 +7
Branches 3267 3270 +3
=======================================
+ Hits 22119 22126 +7
Misses 8757 8757
Partials 111 111
Continue to review full report at Codecov.
|
@@ -114,7 +111,6 @@ function SelectFilter({ | |||
let result = inputValue ? [] : [clearFilterSelect]; | |||
// only call fetch once | |||
// TODO: allow real async search with `inputValue` | |||
if (optionsCache.current) return optionsCache.current; | |||
if (fetchSelects) { | |||
const selectValues = await fetchSelects(inputValue); |
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.
search the current value.
6cb85ff
to
bce9ce7
Compare
bce9ce7
to
257d0d4
Compare
SUMMARY
followup from: #9988
This removes the problematic
LIMIT -1, -1
from the request for all owners. Search is implemented however owners fetch will be limited to 25 records as per FAB defaults.BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
ADDITIONAL INFORMATION