This project is mirrored from https://github.com/metabase/metabase.
Pull mirroring updated .
- Aug 13, 2024
-
-
Phoomparin Mano authored
* add setup commands * fix settings definition * update environment variables for cli * handle instances not being ready * update error messages * add more specific loading messages * loading spinner state * improve error message * use a fixed demo setup token * remove extraneous spinner * update status checks * update container messages * update wait timing * create api keys * extract constants * remove manual steps * Add anonymous tracking + other things. will need to clean up * Modify SDK for better structure * remove line from print.ts * Update webpack.embedding-sdk-cli.config.js back to production * Add types and add quick note * Fix a typo * Add index file, simplify types, use an array * Add safer json parsing * use delay of 100ms between each setup call * Suggestions from review * ensure that cli works * Attempt to fix jest errors * Remove node-fetch from sdk code to hopefully get unit tests working again * add database connection * add connection details handling * refactor asking for database connection info * apply actual database id for syncing schema * fix failing database sync step * allow table selection * create model for each table * handle errors in model creation * fix incorrect model display name * create x-rays based on user data * consolidate instance setup message * workaround for inquirer eventemitter issue * add sample components * fix yarn.lock file * add sample components * set spinner to fail state when instance setup fails * use a transparent background by default * update cli welcome messages * add import notice * refactor imports * update snippet to add dashboard dropdown * cli add credential file to users .gitignore * add docs on quickstart * clear screen with console.clear * address code review feedback Co-authored-by:
Nicolò Pretto <info@npretto.com> * allow node 18 lts --------- Co-authored-by:
Oisin Coveney <oisin@metabase.com> Co-authored-by:
Nicolò Pretto <info@npretto.com>
-
Nemanja Glumac authored
Repro #46756: Add/remove a single column in a model with tables joined by themselves adds/removes all columns (#46757)
-
Uladzimir Havenchyk authored
-
Chris Truter authored
-
Mahatthana (Kelvin) Nomsawadi authored
* Fix conventinoal commit reminder * debug conventional commit reminder * debug conventional commit reminder * Revert readme used to test the workflow
-
Nicolò Pretto authored
-
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)
-