This project is mirrored from https://github.com/metabase/metabase.
Pull mirroring updated .
- Jun 19, 2024
-
-
Kamil Mielnik authored
* Upgrade @reduxjs/toolkit from ^1.9.3 to ^2.2.5 * Fix typing * Upgrade redux-actions and @types/redux-actions * Fix type errors * Fix type errors * Upgrade react-redux from ^8.0.5 to ^9.1.2 - remove @types/react-redux as it's no-longer-needed * Upgrade redux-promise from ^0.5.0 to ^0.6.0 - upgrade @types/redux-promise * Remove unused @types/redux-logger * Revert "Upgrade redux-actions and @types/redux-actions" This reverts commit 65005a00. * Remove createWeakSelector as weakMapMemoize is the default memoization strategy now * Do not call unsubscribe if it does not exist * Do not subscribe * Remove console.log * Make DelayedLoadingSpinner always render something with data-testid="spinner" so that tests can be aware of when things are loading * Fix JoinStep unit tests * Fix AddToDashSelectDashModal unit tests * Fix SegmentApp unit tests * Bring back original subscription logic * Fix DataUsageStep unit tests * Fix ManageApiKeys unit tests * Simplify comment * Fix MainNavbar unit tests * Fix HomeRecentSection unit tests * Use afterEach instead of afterAll to reset mocks * Remove explanatory variable
-
Ngoc Khuat authored
-
Ryan Laurie authored
-
Nemanja Glumac authored
* Rename "empty" date/time filter options * Add repro for #44101
-
Oisin Coveney authored
-
- Jun 18, 2024
-
-
adam-james authored
* Pivot Tables with no Pivot Columns should still download Fixes #44159 Pivot Tables can still be valid without columns, and therefore the download should respect such a query and be successfully exported. In this PR, the postprocessing of pivot results into 'visual pivot' exports for csv takes into account the zero-column configuration and the download no longer fails. * add a test to confirm that zero cols and >1 measures works as well * Handle the case where there are zero pivot-rows. This is a similar problem to the zero pivot-cols case, just with rows. The app actually doesn't render these tables correctly, but that's a separate frontend bug
-
Benoit Vinay authored
* DWH plans added * UploadCSV button component created * Comments added * Main nav bar updates * Misc * Align main nav bar button with link * Misc comment * Upload package created * Various upload cleanup * Misc * Display button rule cleaned up * Tracking added to upload CSV button * Misc comment * e2e tests WIP * More comments * e2e tests basic created * NEw tracking schema * e2e tests updated / Tracking WIP * Analytics cleanup for button * Misc * Misc e2e tests Snowplow * Removed forced Upload CSV button in FE * Upload helpers created * Misc unit tests updates * Updated analytics * Fix for e2e tests * Upload fixture paths updated * Misc for Snowplow * Misc for plan * More tests cleanup * Misc * Source added to analytics * Plan unit tests fixed * Update frontend/src/metabase/nav/containers/MainNavbar/MainNavbarContainer/MainNavbarView.tsx Co-authored-by:
Ryan Laurie <30528226+iethree@users.noreply.github.com> * Update frontend/src/metabase/components/upload/constants.ts Co-authored-by:
Ryan Laurie <30528226+iethree@users.noreply.github.com> * Misc imports * Retrieving `uploads-settings` DB * Remove default UploadCSV component export * Misc * Button copy updated * Analytics updated with optional properties * Default upload file input id updated * csvupload schema updated * Removed before * No bad Snowplow events added --------- Co-authored-by:
Ryan Laurie <30528226+iethree@users.noreply.github.com>
-
Romeo Van Snick authored
* Add RTK endpoints for loading and searching parameters * Add Parameter API types * Move types to metabase-types * Add return type to normalizeQuery
-
Oisin Coveney authored
-
Anton Kulyk authored
* Add repro test * Fix timeline events filtering * Don't include first date of next interval
-
Alexander Polyankin authored
E2E repro for "Fix field values remapping with models & custom dropdown source in public dashboards" (#44315)
-
John Swanson authored
I'd fixed the `can_restore` on non-collections to always be present, but forgot about fixing it for collections as well.
-
Nick Fitzpatrick authored
* use effective location when building initial path in entity picker * unit test adjustment * more unit test adjustments
-
Nemanja Glumac authored
* Convert drill-through (CLJ) values to JS * Convert the drill-through `:null` to an actual `null` Resolves #44232 * Use new function in `filter-drill-details` * display-info->js should check for nil before seqable? * fix js object comparison * Add test for the `drill-value->js` function * Simplify the test * Fix the JS object equality in the test --------- Co-authored-by:
Alexander Solovyov <alexander@solovyov.net>
-
Nemanja Glumac authored
-
Alexander Polyankin authored
-
Alexander Polyankin authored
-
Nemanja Glumac authored
-
Roman Abdulmanov authored
-
Oisin Coveney authored
-
Oisin Coveney authored
-
Ngoc Khuat authored
-
Ngoc Khuat authored
* Writing migration notes
-
- Jun 17, 2024
-
-
Alexander Polyankin authored
-
Nick Fitzpatrick authored
* search events 1-1-0 * backwards compatible adjustment * making context optional
-
Jesse Devaney authored
* refactor: de-duplicate variable usage * fix incorrect goal-line plotting in pow and log y-axis scales * add Loki variants to validate and enforce desired behavior * update loki snapshots
-
Case Nelson authored
* Change run to make-run in process-query-for-card Fixes #44160 There were two levers where a card could change how it was run. `qp` this is basically `qp/process-query` with some optional userland flags and constraints marked on the passed in query. In the case of pivot tables in the `/pivot/:card-id/query` endpoints it was `qp.pivot/process-query` 'run' this defaults to a streaming response that takes the `qp` param and `export-format` and passes a query to qp. Most often this is overriden to not stream the response. Unfortunately, if `run` is overwritten, then `qp` is thrown away, and the caller who overrides `run` must determine the correct `qp` to use. However, when running `process-query-for-dashcard` it is difficult to properly set `qp` for all of the dashcards without doing card lookups. So [this commit](https://github.com/metabase/metabase/blob/release-x.50.x/src/metabase/query_processor/card.clj#L230-L232) did a check to swap the runner in the presence of a pivot query that calls the **default** runner with a `qp.pivot/run-pivot-query`. The problem is that pulse needs to use a non-default runner, but there's no way for `process-query-for-card` to do both. This commit changes `run` to `make-run` which raises the abstraction to a function that produces a runer given a `qp`. This allows `process-query-for-card` to change the `qp` as needed while keeping the passed in runner. There's really three things going on that could be teased apart further/better. 1. `qp` - this conflates the processor (`qp/process-query` `qp.pivot/run-pivot-query`) and modifying query (`qp/userland-query`, `qp/userland-query-with-default-constraints` `(update query :info merge info)`) 2. `make-run` - How the above are processed: (streaming/realized) * Fix tests
-
Alberto Perdomo authored
Tweak the callout about the SDK still being in development
-
Alex Yarosh authored
-
Raphael Krut-Landau authored
* wip * Remove references to the old enable-query-caching setting * remove fixme * linter * remove some more instances * fix imports
-
Alexander Polyankin authored
-
Ryan Laurie authored
* skip super slow xray test * unmark not very slow tests
-
Nemanja Glumac authored
* Elevate include this period in the relative date picker (#44179) * Mock new row with "include {period}" string * Implement "include this" using a toggle * Elevate "starting from" option in the relative date picker (#44242) * Elevate "Starting from" option in the filter picker * Tweak the date offset close icon * Fix tests
-
Jeff Bruemmer authored
-
Ryan Laurie authored
-
- Jun 14, 2024
-
-
Raphael Krut-Landau authored
-
Jesse Devaney authored
* fix column scaling issues in waterfall chart - refactor dataset scaling to behave like scatter and cartesian chart models * add E2E test
-
Raphael Krut-Landau authored
-
Jeff Bruemmer authored
-
Aleksandr Lesnenko authored
* fix charts crash in safari when dataset timezone is not in iana format * Update Loki Snapshots * better fix: apply offset instead of finding a timezone with such offset * remove unused specs * Update Loki Snapshots * review --------- Co-authored-by:
Metabase Automation <github-automation@metabase.com>
-