This project is mirrored from https://github.com/metabase/metabase.
Pull mirroring updated .
- Mar 04, 2024
-
-
Alexander Solovyov authored
-
- Mar 01, 2024
-
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
- Feb 28, 2024
-
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
Removing a trailing `/` Co-authored-by:
Tony-metabase <110378427+Tony-metabase@users.noreply.github.com>
-
- Feb 27, 2024
-
-
Emmad Usmani authored
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
Uladzimir Havenchyk authored
* [Card] Do not pass params with value: null when it's collection preview * return an empty array * Add a test * Fix test * Fix test
-
- Feb 26, 2024
-
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
Kamil Mielnik authored
* Include e2e directory in prettier scripts * Align ESLint & prettier paths in "lint-stage" & "scripts" in package.json * Handle ts files in e2e tests * Format "lint-prettier-pure" script the same way as "prettier" * Ignore database files in ESLint as they reference a file that is git-ignored but does exist locally * Remove duplicated eslint call * Manually fix ESLint-reported issues * Upgrade eslint-plugin-import * Configure import/order rule for consistent, predictable imports * Change "eslint-disable-next-line no-restricted-imports" into "eslint-disable-line" * Ensure imports are in a single block * Move or remove informative comments between import statements * Remove imports-grouping comments * Configure groups - the difference between the default setting is the presence of "internal" group * Manually fix imports order and disable "import/order" rule in files with possible circular dependencies * Move comment to the same line as import * Run "yarn eslint-fix" 5 times * Manually fix imports order in remaining files * Run "yarn prettier" * Fix circular dependency
-
- Feb 22, 2024
-
-
adam-james authored
* WIP flow :semantic-type through QP for aggregations Fixes: #38022 WIP need to add and fix some tests and confirm if this is the most correct place to fix the problem. Seems like it's a good spot to put things though. * Add test asserting that :semantic_type does flow through QP Also fix a couple tests that now have a :semantic_type key. * Add different aggregation types to semantic-type test Working on adding tests and fixing various aggregation semantic types * Test :semantic-type for all aggregations and adjust some metadata-methods * Better use of existing methods via derive * Properly set hierarchy between ::count-aggregation and ::quantity-aggregation
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
Cal Herries authored
Co-authored-by:
Case Nelson <case@metabase.com>
-
- Feb 21, 2024
-
-
adam-james authored
* Do not create param_values on Dashboard Load, only fetch if they exist (#38980) * Do not fetch param_values on Dashboard Load Fixes: #38826 This change removes the *-or-create* behaviour of `field-ids->param-field-values`. This means that on Dashboard Load, if the param values already exist, they'll be grabbed. But, if they don't, we won't try to fetch or create them. This is WIP because it seems to contradict changes related to fixing #30218 * Comment out the other test added for #30218 * Remove the tests from 30218 These tests are asserting that param_values are created on dashboard load when the Field Values don't exist. This is the behaviour we want to revert, so the tests are asserting something we don't want anymore. * Improved test string wording * Update src/metabase/models/params.clj Co-authored-by:
Ngoc Khuat <qn.khuat@gmail.com> --------- Co-authored-by:
Ngoc Khuat <qn.khuat@gmail.com> * Add the refactored fns that were used to solve 38826 * empty commit to try re-start CI --------- Co-authored-by:
Ngoc Khuat <qn.khuat@gmail.com>
-
- Feb 16, 2024
-
-
metamben authored
Fixes #30454 for x.48.x.
-
metabase-bot[bot] authored
* Update test * Remove driver exclusions Co-authored-by:
lbrdnk <lbrdnk@users.noreply.github.com>
-
- Feb 13, 2024
-
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
- Feb 12, 2024
-
-
metabase-bot[bot] authored
- do the unzip sooner, still hash from the file system - java-time -> java-time.api Co-authored-by:
Bryan Maass <bryan.maass@gmail.com>
-
metabase-bot[bot] authored
Sometimes analytics bits get removed by other tests (e.g. deleting all cards, dashboards, users, etc.). When we run `ensure-audit-db-installed!`, though, the checksum hasn't changed (since the underlying analytics files didn't change), so we skip reloading the data. This sets `last-analytics-checksum` to `-1` (meaning: skip the checksum and just load the analytics data) when running `ensure-audit-db-installed!` in the tests that we've seen flake. Co-authored-by:
Bryan Maass <bryan.maass@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Luis Paolini <paoliniluis@gmail.com> Co-authored-by:
Cal Herries <39073188+calherries@users.noreply.github.com>
-
metabase-bot[bot] authored
* load analytics only when changed using a checksum (#38280) * load analytics only when changed using a checksum - over audit deserialization files * update docstrings * fix testing wording * remove :export? It won't get exported because it's not in `metabase.models.setting/exported-settings`. --------- Co-authored-by:
bryan <bryan.maass@gmail.com>
-
metabase-bot[bot] authored
* Fix time-interval against expressions When simplifying time-interval filters, we forgot to place a temporal-unit on expressions, which meant that the date-trunc used by quarters had to match the date exactly rather than being in the interval range. * Update optimize temporal filters for expressions There is a dependency with this middleware and mongo. Without optimization mongo cannot compare the bucketed timestamps because AFAICT mongo 4 doesn't have date truncation so we do so in clojure for the compared literal but cannot do the lhs field/expression. Co-authored-by:
Case Nelson <case@metabase.com> Co-authored-by:
Cal Herries <39073188+calherries@users.noreply.github.com>
-
metabase-bot[bot] authored
* Compare datetimes w/o timezone fields with literals as strings Co-authored-by:
metamben <103100869+metamben@users.noreply.github.com>
-
metabase-bot[bot] authored
* Fix custom destination to public dashboard not working on static embedding (#38647) * Fix linking to public or embed resource on dashboards not working * Update embedding test name to use a snazzier term * Add tests * Add missing test util from 49 --------- Co-authored-by:
Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev> Co-authored-by:
Mahatthana Nomsawadi <mahatthana.n@gmail.com>
-
- Feb 09, 2024
-
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Thomas Schmidt <somtom91@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Cal Herries <39073188+calherries@users.noreply.github.com>
-
metabase-bot[bot] authored
Co-authored-by:
Chris Truter <crisptrutski@users.noreply.github.com>
-
- Feb 07, 2024
-
-
metabase-bot[bot] authored
* Update default Metabot model The default GPT-based Metabot model has been `"gpt-4"`. This model is no longer state of the art, has a limit of 8,192 tokens, and is priced at input: $0.03/ktoken and output: $0.06/ktoken. `"gpt-4-turbo-preview"`, which points internally to `"gpt-4-0125-preview"` is the current state of the art, has a limit of 128,000 tokens, and is priced at input: $0.01/ktoken and output: $0.03/ktoken. This should solve any issues of reasonable cases of token limit failures (e.g. [Metabot 2.0 failing when context length exceeded](https://github.com/metabase/metabase/issues/29932)) without needing to do any extra handling on the backend. For more info on the models, token limits, and pricing, see: - [OpenAI model docs](https://platform.openai.com/docs/models/gpt-4-and-gpt-4-turbo) - [OpenAI pricing](https://openai.com/pricing ) * whitespace * Removing irrelevant settings test Co-authored-by:
Mark Bastian <markbastian@gmail.com>
-
metabase-bot[bot] authored
* Fix ESlint rules (#38458) * Fix ESlint from the result of upgrading ESLint package * Support `component` prop on Mantine * Prevent accidental ESLint log in Cypress * Upgrade ESLint rule * Fix ESLint --------- Co-authored-by:
Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev> Co-authored-by:
Mahatthana Nomsawadi <mahatthana.n@gmail.com>
-
metabase-bot[bot] authored
add a comment about enterprise version Co-authored-by:
Luis Paolini <paoliniluis@gmail.com>
-
- Feb 06, 2024
-
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
adam-james authored
* Add internal setting for table render row limit (#36996) * Add internal setting for table render row limit This change will keep default behaviour of a 10 row limit but allow users to set an env variable to change this default. * test that table render row limits can be adjusted w/ env var. Add a test that shows the table render respects the default 10 row limit, and also show that if the user uses the environment variable `MB_ATTACHMENT_TABLE_ROW_LIMIT`, then they can change the row limit. * Explicitly unset env. var in the test so it truly tests the default * Adjust :getter for setting so that defaults are used appropriately * Fix tests that expected exact text matches on some hiccup nodes * New default of 20. * Add a limit of 100 to the rows and add a test for it too. * Keep rows-limit def but set it to use the public-setting * Fix up settings tests
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
Co-authored-by:
Jeff Bruemmer <jeff.bruemmer@gmail.com>
-
metabase-bot[bot] authored
[E2E] Overload the existing E2E `visitDashboard` helper to make it work with the alias (#38309) (#38473) * Add `visitDashboardAlias` E2E helper * Overload `visitDashboard` helper It should work with either dashboard id, or with a dashboard id alias. * PoC: Use overloaded `visitDashboard` helper * Update even more tests with the overloaded `visitDashboard` helper * Use `cy.visit` for tests with `cy.clock` instead of a custom helper * Rename non-exported functions Co-authored-by:
Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>
-
metabase-bot[bot] authored
Adding `r.metadata.source.branch === "master" &&` to `randomlyUploadAll` for master-only uploads. Also adding more info to log message. (cherry picked from commit ab9ede8e) Co-authored-by:
Mark Bastian <markbastian@gmail.com>
-