forked from metabase/metabase
-
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
update metabase #10
Open
luizbob
wants to merge
10,000
commits into
Send4BR:staging
Choose a base branch
from
metabase:master
base: staging
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
update metabase #10
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
* add Dan's mb-download script * add short bin/README.md
…52054) * Remove the Cypress "version check" step * Remove superfluous step from GH Actions workflows
* Update mysql.md Add an additional flag as a note, since I've hit this one today with a customer * Update docs/databases/connections/mysql.md Co-authored-by: Jeff Bruemmer <[email protected]> --------- Co-authored-by: Jeff Bruemmer <[email protected]>
This reverts commit bf88842.
* Enabling mouse wheel scroll zooming for leaflet maps * Reducing wheelPxPerZoomLevel so scroll zooming works faster in leaflet maps
…e and positive values (#51941)
* WIP First attempt at passing start of week option to static viz bundle * Cleaning debug lines * set moment and dayjs start of week day setting in static viz * reverting index.ts * Adding backend test for checking first day of week setting is respected in static viz --------- Co-authored-by: Aleksandr Lesnenko <[email protected]> Co-authored-by: Aleksandr Lesnenko <[email protected]>
* Metabase `core` module cleanup * Fix some references
* Remove remaining MetaBot v2 code * Remove reference to metabot v2 mocks
…#49714) * [debug] add storybook helper * Make error on InteractiveQuestion consistent * Make error on StaticQuestion consistent * [debug] static question stories * [debug] InteractiveDashboard Stories * Make error on InteractiveDashboard consistent * [debug] StaticDashboard stories * Make error on StaticDashboard consistent * [debug] EditableDashboard stories * Make error on EditableDashboard consistent * Fix CreateQuestion not working after fixing entity ID states * Add tests for InteractiveQuestion * Add StaticQuestion tests * Add InteractiveDashboard tests * Add StaticDashboard tests * Add EditableDashboard tests * Make variable names consistent * Extract resource not found error to a component * Fix missing return * Make storybook usable and reduce duplications * Don't roll my own util * Update error colors following the new design * Update loki snapshots * Fix unit test expectation to hex * Make colors easier to understand with comments * Update SDK error style * Update error components copy * Use Code from metabase/ui * Remove change accidentally commited * Fix InteractiveQuestion renders question not found briefly #51969
* Remove `metabase.troubleshooting` * Remove Kondo config
* [Backend modularization] Remove `metabase.compatibility` * Fix one more ref
* [Backend modularization] `analyze` module cleanup * Appease Kondo
This change happened in peer #51023 and is manifesting now it seems. The removal of the call to `(classloader/the-classloader)` is what is hurting us. Only clickhouse cloud instances can be set as a suitable upload spot so i'm doing the following changes in the local jar repro to fix it up: ```clojure (in-ns 'metabase.driver.clickhouse) (defmethod driver/database-supports? [:clickhouse :uploads] [_driver _feature db] true) (in-ns 'metabase.public-settings) (let [db_id 2, schema_name "default", table_prefix "upload_"] (t2/update! :model/Database db_id {:uploads_enabled true :uploads_schema_name schema_name :uploads_table_prefix table_prefix})) ```
* [Backend modularization] `query-analysis` cleanup * Update Kondo config * Add CODEOWNERS * metabase.task.sweep-query-analysis => metabase.query-analysis.task.sweep-query-analysis * Move test util * Module linter should ignore my test-setup namespace * Make sure query analysis init is loaded in tests that need it * Deparallelize some of the tests
* Revise new table permissions logic to handle sandboxes * tweak new perm logic * make sure cond has a default * fix comments * tweak defenterprise conditions
* Add ericnormand to query team and slack map * Fix whitespace * Remove trailing whitespace
* allow passing aggregation instead of expression * add e2e test
Closes: #52116 Note that of icu4j was already a dependency, via graalvm. We have just made it explicit now, ~~and updated to the latest version.~~ ([upgrading version breaks rendering](https://github.com/metabase/metabase/actions/runs/12926546646/job/36049844305?pr=52581)) ``` org.graalvm.js/js 22.3.5 . org.graalvm.regex/regex 22.3.5 . org.graalvm.truffle/truffle-api 22.3.5 . com.ibm.icu/icu4j 71.1 . org.graalvm.truffle/truffle-api 22.3.5 . org.graalvm.sdk/graal-sdk 22.3.5 . org.graalvm.sdk/graal-sdk 22.3.5 . com.ibm.icu/icu4j 71.1 ```
* chore: upgrade undici package with a security fix * remove resolution for source-map-js
* throw down ui * pixel freaking perfect * wire in real endpoint for fetching dashboard question candidates * wire in move endpoint * loading and error handling for mutation * persist info step acknowledgement * optionally show item in menu, fix flickering of info modal is acked previously, add a toast on successful bulk move * fix some of the failing existing tests * add unit tests for collection menu * add unit tests for main migration modal * add e2e test * fix failing unit test * fix another failing unit test * fix candidates result being stale if dashcards added/removed from dashboards creates/removes candidates * adjust url of feature * copy feedback * fix type-check * scope feature to only admins * use pagination to fetch count but not items when detecting if to show the migration tool menu item * test + style improvements, impl pr feedback * fix unit tests, add unit coverage for is admin
…g a dashboard question (#52406) * adds back prop that prevented feature from working * fix e2e
* mock time using with-redefs * remove parallel * use with-clock
* Add caveat about SDK client filtering Co-authored-by: Jeff Bruemmer <[email protected]>
* Remove &request anaphor and make it a normal arg * Add test * Defendpoint 2.1 * base-handler => handler * Fix Kondo errors
* Auto-rewrite more defendpoints * Test fixes 🔧 * Test fixes 🔧
* Drop Java 11/17 Support * Update Dockerfile Change the Java version of the builder as well * mention that we are dropping support for earlier Java in docs --------- Co-authored-by: Luis Paolini <[email protected]> Co-authored-by: Alexander Solovyov <[email protected]>
Co-authored-by: Noah Moss <[email protected]>
* Make preemptive caching toggle a FE plugin * make sure we expose the premium feature to the FE * fix BE test
* Auto-rewrite sandboxing REST API endpoints * attribute_remappings should be optional * Test fix 🔧
* Remove defendpoint-async * Rewrite all the rest of the remaining defendpoints except for those in public * Remove card endpoints from this PR * Update src/metabase/setup/api.clj Co-authored-by: bryan <[email protected]> * cljfmt * Test fixes 🔧 --------- Co-authored-by: bryan <[email protected]>
* defendpoint 2.2 * Remove unused stuff * PR feedback from @escherize
* Auto-rewrite defendpoint forms in session.clj * Auto-rewrite defendpoint forms in session.clj * Revert changes to session * Auto-rewrite endpoints in table.clj * Restore session.clj * OK unrestore session * Fix macroexpansion * Support non-map HTTP request bodies * Remove &request anaphor and make it a normal arg * Add test * Defendpoint 2.1 * base-handler => handler * Fix Kondo errors * Update src/metabase/api/table.clj Co-authored-by: bryan <[email protected]> * Update src/metabase/api/table.clj Co-authored-by: bryan <[email protected]> * no more `&request` --------- Co-authored-by: bryan <[email protected]>
* Auto-rewrite defendpoint forms in session.clj * Auto-rewrite defendpoint forms in session.clj * Remove &request anaphor and make it a normal arg * Add test * Defendpoint 2.1 * base-handler => handler * Fix Kondo errors * Make sure async exception handling is done correctly
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.
Before submitting the PR, please make sure you do the following
Tests
yarn lint && yarn test
)clojure -X:dev:test
(unless it's a tiny documentation change).