fix: Adds a loading message when needed in the Select component #16531

michael-s-molina · 2021-08-31T19:41:10Z

SUMMARY

Adds a loading message when needed in the Select component. Previously, we displayed the 'No data' message while the select was still loading the items.

AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2021-08-31.at.4.24.30.PM.mov

TESTING INSTRUCTIONS

Open the Select Storybook
Check for the loading indicator while applying different combinations of the controls

ADDITIONAL INFORMATION

Has associated issue:
Required feature flags:
Changes UI
Includes DB Migration (follow approval process in SIP-59)
- Migration is atomic, supports rollback & is backwards-compatible
- Confirm DB migration upgrade and downgrade tested
- Runtime estimates and downtime expectations provided
Introduces new feature or API
Removes existing feature or API

codecov · 2021-08-31T20:08:58Z

Codecov Report

Merging #16531 (f171694) into master (c5a5cf7) will decrease coverage by 0.00%.
The diff coverage is 36.36%.

@@            Coverage Diff             @@
##           master   #16531      +/-   ##
==========================================
- Coverage   76.71%   76.70%   -0.01%     
==========================================
  Files        1002     1002              
  Lines       53784    53798      +14     
  Branches     6859     6862       +3     
==========================================
+ Hits        41259    41267       +8     
- Misses      12288    12294       +6     
  Partials      237      237

Flag	Coverage Δ
javascript	`70.99% <36.36%> (-0.01%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset-frontend/src/components/Select/Select.tsx	`72.45% <36.36%> (-0.97%)`	⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5a5cf7...f171694. Read the comment docs.

villebro

Code LGTM, this will make the Select UX much better! ❤️ Also looking forward to the tests, as the complexity is adding up here.

michael-s-molina · 2021-09-02T11:21:58Z

Code LGTM, this will make the Select UX much better! ❤️ Also looking forward to the tests, as the complexity is adding up here.

Thanks for the review! The tests are already being cooked! 🧑‍🍳

geido

LGTM! Thanks for this!

…he#16531)

* add support for adhoc columns to api and sqla model * fix some types * fix duplicates in column names * fix more lint * fix schema and dedup * clean up some logic * first pass at fixing viz.py * Add frontend support for adhoc columns * Add title edit * Fix showing custom title * Use column name as default value in sql editor * fix: Adds a loading message when needed in the Select component (#16531) * fix(tests): make parquet select deterministic with order by (#16570) * bump emotion to help with cache clobbering (#16559) * fix: Support Jinja template functions in global async queries (#16412) * Support Jinja template functions in async queries * Pylint * Add tests for async tasks * Remove redundant has_request_context check * fix: impersonate user label/tooltip (#16573) * docs: update for small typos (#16568) * feat: Add Aurora Data API engine spec (#16535) * feat: Add Aurora Data API engine spec * Fix lint * refactor: sql_json view endpoint: encapsulate ctas parameters (#16548) * refactor sql_json view endpoint: encapsulate ctas parameters * fix failed tests * fix failed tests and ci issues * refactor sql_json view endpoint: separate concern into ad hod method (#16595) * feat: Experimental cross-filter plugins (#16594) * fix:fix get permission function * feat: add cross filter chart in charts gallery under FF * chore(deps): bump superset-ui to 0.18.2 (#16601) * update type guard references * fix imports * update series_columns schema * Add changes that got lost in rebase * Use current columns name or expression as sql editor init value * add integration test and do minor fixes * Bump superset-ui * fix linting issue * bump superset-ui to 0.18.22 * resolve merge conflict * lint * fix select filter infinite loop * bump superset-ui to 0.18.23 * Fix auto setting column popover title * Enable adhoc columns only if UX_BETA enabled * put back removed test * Move popover height and width to constants * Refactor big ternary expression Co-authored-by: Kamil Gabryjelski <kamil.gabryjelski@gmail.com> Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com> Co-authored-by: Elizabeth Thompson <eschutho@gmail.com> Co-authored-by: Rob DiCiuccio <rob.diciuccio@gmail.com> Co-authored-by: Beto Dealmeida <roberto@dealmeida.net> Co-authored-by: joeADSP <75027008+joeADSP@users.noreply.github.com> Co-authored-by: ofekisr <35701650+ofekisr@users.noreply.github.com> Co-authored-by: simcha90 <56388545+simcha90@users.noreply.github.com>

…he#16531)

* add support for adhoc columns to api and sqla model * fix some types * fix duplicates in column names * fix more lint * fix schema and dedup * clean up some logic * first pass at fixing viz.py * Add frontend support for adhoc columns * Add title edit * Fix showing custom title * Use column name as default value in sql editor * fix: Adds a loading message when needed in the Select component (#16531) * fix(tests): make parquet select deterministic with order by (#16570) * bump emotion to help with cache clobbering (#16559) * fix: Support Jinja template functions in global async queries (#16412) * Support Jinja template functions in async queries * Pylint * Add tests for async tasks * Remove redundant has_request_context check * fix: impersonate user label/tooltip (#16573) * docs: update for small typos (#16568) * feat: Add Aurora Data API engine spec (#16535) * feat: Add Aurora Data API engine spec * Fix lint * refactor: sql_json view endpoint: encapsulate ctas parameters (#16548) * refactor sql_json view endpoint: encapsulate ctas parameters * fix failed tests * fix failed tests and ci issues * refactor sql_json view endpoint: separate concern into ad hod method (#16595) * feat: Experimental cross-filter plugins (#16594) * fix:fix get permission function * feat: add cross filter chart in charts gallery under FF * chore(deps): bump superset-ui to 0.18.2 (#16601) * update type guard references * fix imports * update series_columns schema * Add changes that got lost in rebase * Use current columns name or expression as sql editor init value * add integration test and do minor fixes * Bump superset-ui * fix linting issue * bump superset-ui to 0.18.22 * resolve merge conflict * lint * fix select filter infinite loop * bump superset-ui to 0.18.23 * Fix auto setting column popover title * Enable adhoc columns only if UX_BETA enabled * put back removed test * Move popover height and width to constants * Refactor big ternary expression Co-authored-by: Kamil Gabryjelski <kamil.gabryjelski@gmail.com> Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com> Co-authored-by: Elizabeth Thompson <eschutho@gmail.com> Co-authored-by: Rob DiCiuccio <rob.diciuccio@gmail.com> Co-authored-by: Beto Dealmeida <roberto@dealmeida.net> Co-authored-by: joeADSP <75027008+joeADSP@users.noreply.github.com> Co-authored-by: ofekisr <35701650+ofekisr@users.noreply.github.com> Co-authored-by: simcha90 <56388545+simcha90@users.noreply.github.com>

fix: Adds a loading message when needed in the Select component

f171694

michael-s-molina requested a review from geido August 31, 2021 19:41

pull-request-size bot added the size/L label Aug 31, 2021

michael-s-molina mentioned this pull request Sep 1, 2021

refactor: Changes the DatabaseSelector and TableSelector to use the new Select component #16483

Merged

9 tasks

michael-s-molina requested a review from villebro September 1, 2021 16:57

villebro approved these changes Sep 2, 2021

View reviewed changes

geido approved these changes Sep 2, 2021

View reviewed changes

michael-s-molina merged commit 02798a3 into apache:master Sep 2, 2021

villebro pushed a commit to preset-io/superset that referenced this pull request Sep 6, 2021

fix: Adds a loading message when needed in the Select component (apac…

d5a655c

…he#16531)

opus-42 pushed a commit to opus-42/incubator-superset that referenced this pull request Nov 14, 2021

fix: Adds a loading message when needed in the Select component (apac…

e83de04

…he#16531)

QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 28, 2021

fix: Adds a loading message when needed in the Select component (apac…

6ee2fdb

…he#16531)

ktmud mentioned this pull request Feb 18, 2022

feat: remove loading indicator when typing in select #18799

Merged

9 tasks

mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.4.0 labels Mar 13, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: Adds a loading message when needed in the Select component #16531

fix: Adds a loading message when needed in the Select component #16531

michael-s-molina commented Aug 31, 2021

codecov bot commented Aug 31, 2021

villebro left a comment

michael-s-molina commented Sep 2, 2021

geido left a comment

fix: Adds a loading message when needed in the Select component #16531

fix: Adds a loading message when needed in the Select component #16531

Conversation

michael-s-molina commented Aug 31, 2021

SUMMARY

AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

codecov bot commented Aug 31, 2021

Codecov Report

villebro left a comment

Choose a reason for hiding this comment

michael-s-molina commented Sep 2, 2021

geido left a comment

Choose a reason for hiding this comment