-
Notifications
You must be signed in to change notification settings - Fork 14k
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
chore: Moves CheckboxIcons to Checkbox folder #13459
chore: Moves CheckboxIcons to Checkbox folder #13459
Conversation
Codecov Report
@@ Coverage Diff @@
## master #13459 +/- ##
==========================================
- Coverage 76.96% 72.48% -4.49%
==========================================
Files 930 633 -297
Lines 46278 21860 -24418
Branches 5554 5708 +154
==========================================
- Hits 35619 15845 -19774
+ Misses 10525 5881 -4644
Partials 134 134
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.
LGTM with a small nit
superset-frontend/src/dashboard/components/filterscope/treeIcons.jsx
Outdated
Show resolved
Hide resolved
114c5c6
to
1c01b2a
Compare
/testenv up |
@rusackas Ephemeral environment spinning up at http://52.10.134.160:8080. Credentials are |
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.
LGTM!
/testenv down |
Ephemeral environment shutdown and build artifacts deleted. |
* master: (56 commits) test: Adds tests and storybook to CertifiedIcon component (#13457) chore: Moves CheckboxIcons to Checkbox folder (#13459) chore: Removes Popover duplication (#13462) build(deps): bump elliptic from 6.5.3 to 6.5.4 in /docs (#13527) fix: allow spaces in DB names (#13800) chore: Update PR template for SIP-59 DB migrations process (#13855) Add CODEOWNERS (#13759) feat(alerts & reports): Easier to read execution logs (#13752) fix: Disallows negative options remaining (#13749) Fix broken link (#13861) fix(native-filters): add global async query support to native filters (#13837) Displays row limit warning with Alert component (#13854) fix(errors): Downgrade error on stop query to a warning (#13826) fix(alerts and reports): Unify timestamp format on execution log view (#13718) fix(sqllab): warning message when rows limited (#13841) chore: add success log whenever a connection is working (#13811) fix(native-filters): improve loading styles for filter component (#13794) chore: update change log with cherry-picks for release 1.1 (#13824) feat: added support to configure the default explorer viz (#13610) fix(#13734): Properly escape special characters in CSV output (#13735) ...
SUMMARY
Moves
CheckboxIcons
toCheckbox
folder.TEST PLAN
1 - Execute all tests.
2 - All tests should pass.
@rusackas @junlincc
ADDITIONAL INFORMATION