This project is mirrored from https://github.com/metabase/metabase.
Pull mirroring updated .
- Aug 13, 2024
-
-
Nicolò Pretto authored
-
Nicolò Pretto authored
fix(sdk): make sure generated password has at least one number, one upper case and one lower case character to avoid issues with password policies (#46737)
-
Mahatthana (Kelvin) Nomsawadi authored
* Fix tooltip styles leaking to popover * Fix snapshot now showing the tooltips and dropdowns * Update the loki snapshot using built storybook for consistency
-
Nemanja Glumac authored
-
Nemanja Glumac authored
* Prevent opening data source in a new tab for embedding SDK * Add unit tests for embedding SDK context
-
- Aug 12, 2024
-
-
Ryan Laurie authored
-
Nemanja Glumac authored
-
Alexander Polyankin authored
-
appleby authored
* Add appleby to Query Processor team in team.json * Add appleby to github-slack-map.json
-
Nemanja Glumac authored
* Show new model loading state while fetching * Show new model loading state while fetching * Add unit test * Use `DelayedLoadingSpinner` instead * Update the test Co-authored-by:
Kamil Mielnik <kamil@kamilmielnik.com> * Prettier --------- Co-authored-by:
Kamil Mielnik <kamil@kamilmielnik.com>
-
adam-james authored
* Add Timeseries insights even when Custom Columns exist WIP Fixes 46244 I don't know why it was (maybe still is) required that the :other columns be empty before providing insights. * might be a solution to changing trendlines when data doesn't change The insights code that powers the frontend trendline requires taking a sample of the dataset. This sampling has some randomness associated by design, and should maintain its randomness. But it makes some sense to use the exact same sample when the input has not changed at all, hence trying this memoize approach out. Might not be the final solution, but it's a start * add test to show that insights are computed * use java.util.random with a seed to keep insights stable
-
Oisin Coveney authored
-
Duncan Mak authored
* Add testcase from https://github.com/metabase/metabase/issues/43915 * Try a custom formatter if the default formatter for OffsetDateTime doesn't work * Add testcase * Make auxillary formatter private * Catch a more specific exception * Throw IllegalArgumentException on invalid dates * Use strict resolver and fix test * Empty commit to assign credit where it's due --------- Co-authored-by:
Chris Truter <chris@metabase.com>
-
adam-james authored
* Dowload Row Limit Env Var Adds `MB_DOWNLOAD_ROW_LIMIT` to enable changing the row limit on downloads and alert/subscription attachments (not the rendered tables, but the .csv, etc.). Based on: #44982 (Thanks, @r-kot) Partially implements: #28144 The difference in this PR compared to #44982 is that the download limit applies to all downloads, the only exception being when the limit is above `qp.i/absolute-max-results` (1048575, based on Excel's limitation); in such a case, the user supplied limit is only used if the download is csv or json, and `qp.i/absolute-max-results` is used for xlsx. This PR also fixes alert/subscription attachment limits; prior to this, they were set to the in-app limit of 2000 rows, but now they will follow the user supplied download-row-limit. This PR also adds a test to the downloads-and-exports test namespace, confirming that they follow the supplied limit, or the max limit if none is supplied. * add test confirming the default limit works * fix test to use the download-row-limit * address review feedback * Update src/metabase/public_settings.clj Co-authored-by:
Cal Herries <39073188+calherries@users.noreply.github.com> * add test covering case where download-row-limit is unset --------- Co-authored-by:
Cal Herries <39073188+calherries@users.noreply.github.com>
-
Uladzimir Havenchyk authored
-
Kamil Mielnik authored
* Refactor filter utils to TS * Remove type casts * Improve dimension instantiation * Fix typing
-
Chris Truter authored
-
Denis Berezin authored
-
Uladzimir Havenchyk authored
-
Alexander Solovyov authored
-
Nick Fitzpatrick authored
* Revert "Remove unused humanizeCoercionStrategy and its test suite (#36136)" This reverts commit 5375347e. * humanize coercion strategies, add e2e test * unit test adjustment * pr feedback
-
Uladzimir Havenchyk authored
* dx: raise an error if lodash is used * fix lint errors * use ts-deepmerge * don't throw an error * use correct type * use merge form icepick
-
Chris Truter authored
-
Alexander Solovyov authored
-
Oisin Coveney authored
-
Oisin Coveney authored
-
Ngoc Khuat authored
-
- Aug 10, 2024
-
-
Oleksandr Yakushev authored
-
Aleksandr Lesnenko authored
* conditional dismissal step * check for download status in the container * try fixing flakes again * try again * chartPathWithFillColor already verifies chart has been rendered
-
- Aug 09, 2024
-
-
Nemanja Glumac authored
* Introduce subpath-safe URL utils * Open data source links from a notebook safely on a subpath * Open browse model links safely on a subpath * Uniform naming * Use the existing URL helper * Trim url * Add unit tests * Add one more test case * Fix test name
* Make URLs non-optional strings -
adam-james authored
* Export Formatting Correctness WIP Several bugs exist around the correctness of export formatting, this aims to fix them. Fixes #27374 When a model has viz settings/metadata changed after save, the download won't incorporate said changes. At least it's clear that scale doesn't work. * number formatter merges settings from columns too * column formatting on aggregated cols works in xlsx now too * fixes 43040 by grabbing global-type-settings in csv formatter
-
github-automation-metabase authored
-
Raphael Krut-Landau authored
fix(admin/performance): Ensure that the tab's title (e.g. Database caching) and 'Performance' are visible in document title (#46602)
-
Oleksandr Yakushev authored
-
Oisin Coveney authored
-
Mahatthana (Kelvin) Nomsawadi authored
* Refactor * Fix wrong colors when passing empty SDK `colors` * Clean up CSS custom properties * Add missing color mapping to the SDK * Add missing colors to the main app * Only override additional dynamic colors when values are passed * Fix unit tests * Address review Co-authored-by:
Phoomparin Mano <poom@metabase.com> --------- Co-authored-by:
Phoomparin Mano <poom@metabase.com>
-
Uladzimir Havenchyk authored
* dx: get rid of csp errors in TableInteractive * get rid of csp errors in mantine
-
Romeo Van Snick authored
Move the compare to the past aggregation to the bottom of the dropdown and hide it when no temporal breakouts are available (#46500) * Move the compare to the past aggregation to the bottom of the dropdown * Fix missing icon property * Hide 'Compare to the past' when there are no temporal columns * Update copy in tests * Add test for questions with no temporal columns * Fix unit test for AggregationPicker * Allow adding a compare aggregation when the first breakout is not temporal too * Rename to 'Compare to the past' in custom column action * Rename to 'Compare to the past' in column drill * Update copy in tests for actions and drills * Move canAddTemporalAggregation to CompareAggregation utils * Use aggregations from the query always * Remove unnecessary displayName check
-
- Aug 08, 2024
-
-
adam-james authored
* Last Used Param Value doesn't fill in with default value if cleared Fixes #46368 The Last used parameter value is cleared when the frontend sends `nil` as the parameter's value. This works except in cases where a user clears a parameter with a default value. In this case, we want to indicate that the parameter is cleared, so we store `nil` in the appdb in such a case. * Update src/metabase/query_processor/dashboard.clj Co-authored-by:
metamben <103100869+metamben@users.noreply.github.com> * add assertion showing that 'nil' with default is still stored --------- Co-authored-by:
metamben <103100869+metamben@users.noreply.github.com>
-
Ryan Laurie authored
* log full version history * remove test code
-