forked from apache/superset
-
Notifications
You must be signed in to change notification settings - Fork 0
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(report): fix last_eval_dttm sort and more tests (apache#12121) … #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* fix(report): fix last_eval_dttm sort and more tests * remove unnecessary permissions and split code path * remove SIP_34_ALERTS_UI * disabling an alert that is working will turn it to not triggered
* fix: database alpha permissions * add test
* Fix anchor-is-valid lint warnings * Change IconTooltip to use named exports
* Refresh interval * WIP
* Change chart_is_fav to chart_is_favorite * Trigger cypress tests * Retrigger cypress tests * Retrigger all tests * Dashboard * Additions
* fix(explore): save to dashboard not working * fix test
* Add Loading icon to Filter Bar * Change Loading Box height to gridUnits
…12174) * fix(reports): notification without standalone link back to superset * fix comment
* docs: update UPDATING.md with security simplification info * update docs better readability
* fix(alerts/reports): ensure correct type is sent to api * fix: chane actions labels to generic Edit/Delete * fix: ts-ignore * fix: add more dependencies to callback functions * fix: remove duplicate api calls when toggling between types * lint
* fix(reports): make name unique between alerts and reports * add missing migration * make it work for mySQL and PG only (yet) * fixing sqlite crazy unique drop * fixing sqlite missing one col
* build: inline cached-dependencies to unblock CI * Run E2E on pull_request on;y * Inline all external actions * Checkout needed for internal actions Also fixes pre-commit * Add missing files
* alert/report modal improvements * update spec for alert report modal
* WIP * Add unit tests for sql_parse * Add unit tests for sql_lab
Co-authored-by: Bryan Keller <bkeller@netflix.com>
* fix bar chart order * fix test_explore_json_dist_bar_order * fix quotes
* fix: various style touch on Explore page * More style fixes * Force 100% height for sidebars * Fix linting
The connection prefix should be different if the DB2 dialect is different.
* feat: add decorator to guard public APIs * Add unit tests * Refactor to use unit tests * Paramterize tests * Remove decorator
1. Upgrade match-sorter from 4.1.0 to 6.1.0 2. Add a debounce delay of 200 milliseconds to reduce excessive rendering (and searching) 3. Set keepDiacritics to true to improve performance 4. Display count of filtered results in "Showing x of xx", instead of the total results 5. Rank certified metrics to the top
* fix(load_examples): better fix for load_data * Address changes
* Show incompatible native filters indicator * Add Native Filters mocks and tests to Filter Badge * Compare filter names in deduplication logic * Add indicator key * Remove unnecessary import
* Fix UI for long native filters names * Add Filter width const * Refactor Filter Bar and Config Modal * Break word for long Filter Control titles
* Reset column field for removed dataset * Fix issue with cleaning exisiting field * Fix lint error and remove unnecessary import * Change message text for wrong filter * Add Basic Error Alert component * Fix hiding error after filter correction
* fix(multiline): return chart data on data request * bump package * optimize chart retrieval and fix chart form_data
#12709) * Use git submodules for (securely) using third party Github Actions List of repositories added as submodules: EndBug/latest-tag@latest morrisoncole/pr-lint-action@v1.4.1 trilom/file-changes-action@v1.2.4 styfle/cancel-workflow-action@0.6.0 apache-superset/cached-dependencies@b90713b unsplash/comment-on-pr@v1.2.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix(report): fix last_eval_dttm sort and more tests (apache#12121) …