-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
feat: Add toast for successful table or schema refreshes in Sqllab #18169
Conversation
Codecov Report
@@ Coverage Diff @@
## master #18169 +/- ##
==========================================
- Coverage 65.99% 65.96% -0.03%
==========================================
Files 1587 1590 +3
Lines 62176 62227 +51
Branches 6273 6285 +12
==========================================
+ Hits 41032 41050 +18
- Misses 19523 19555 +32
- Partials 1621 1622 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
One small console log.
/testenv up |
@pkdotson Ephemeral environment spinning up at http://34.215.214.67:8080. Credentials are |
Ephemeral environment shutdown and build artifacts deleted. |
…pache#18169) * save * add use redux to test * update test for db selector * add condition for refresh only
…pache#18169) * save * add use redux to test * update test for db selector * add condition for refresh only
…pache#18169) * save * add use redux to test * update test for db selector * add condition for refresh only
SUMMARY
Before users weren't notified if the table or schemas were properly refreshed. So adding toast to inform the users on successful state of refresh.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Screen.Recording.2022-01-25.at.3.23.59.PM.mov
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION