This project is mirrored from https://github.com/metabase/metabase.
Pull mirroring updated .
- May 12, 2023
-
-
Jeff Bruemmer authored
* add links to paid features section * add readme link * update links
-
bryan authored
* add column for marking the audit database * create audit db on app startup On app startup, in enterprise mode, always check that there is an up-to-date audit database with is_audit set to true. up-to-date means that it matches the app-db. If the audit db doesnt exist, or doesn't match up with the app-db, it will be created. otherwise no-op - refactor GET /api/database/ into defendpoint * add a few tests * add more tests + return kw for no-op * Merge branch 'master' into bcm-add-app-db-as-audit-db * fix migrations file * lint fix + remove def * keep the old audit db around - nil -> ::no-op * small macro fix * Merge branch 'master' into bcm-add-app-db-as-audit-db * Merge branch 'master' into bcm-add-app-db-as-audit-db * move ensure-db-exists! into its own namespace - avoids circlar dependency on app startup * fix tests * lint fix * inspect test values * use mdb.env/env to initialize audit db * try returning the app-db connection pool - instead of making a doppelganger in metabase database * get the is_audit column when looking up dbs in qp - this key is needed to return the appdb for the "Audit DB"'s connection * clean up audit-db install/update/noop logic * remove audit db from GET api/database/ * Merge branch 'master' of github.com:metabase/metabase into bcm-add-app-db-as-audit-db * fix linter + inline extra function * fix a test * Merge branch 'master' into bcm-add-app-db-as-audit-db * don't test actions on audit db
* Revert "don't test actions on audit db " This reverts commit 581ed1ab9495de7b46bccb8b1ab19f66a8fd106d. * for tests to work low IDs are basically reserved - use a higher id when installing audit db * use defenterprise for ensure-audit-db-installed! * Merge branch 'master' into bcm-add-app-db-as-audit-db -
Natalie authored
-
Natalie authored
-
Ngoc Khuat authored
* Dashboard Tab entity (#29802) * add dashboard tab entity * Dashboard tabs CRUD (#29861) * Dashboard tabs frontend (#30189) * rename `DashboardHeader.tsx` to `DashboardHeaderView` to avoid collision with `DashboardHeader.jsx` commit-id:7cdfd86a * add tabs to dash commit-id:d3c2fa74 * improve code structure and other fixes * rename `dashboardtab_id` to `dashboard_tab_id` * remove explicit `position` logic * improve code quality * add renaming functionality * add two new tabs when creating first tab * hide tabs if there is only one * add unit test for `DashboardTabs` component * refactor * fix type errors * add `tabId` to text cards * add `tabId` to link cards * add `tabId` to action button * use disabled state when only one tab remains * refetch cards with filter value when changing tabs * add e2e test * fix broken e2e tests * add horizontal scrolling * fix unit test * fix create tab button being on right * fix performance issue when switching tabs * add `ttag` to `tabs.ts` * use `aria-label` in e2e test helper * fix type error after rebasing * making ordered_tabs optional for tests purposes * fix failing unit tests * fix failing e2e tests * fix flaky revision history e2e test * fix type error after merging feature branch with master and rebasing * fix revisions unit test * fix `ActionParmatersInputsForm.tsx` to stop `actions-on-dashboards.cy.spec.js` from flaking --------- Co-authored-by:
Ngoc Khuat <qn.khuat@gmail.com> --------- Co-authored-by:
Emmad Usmani <emmadusmani@berkeley.edu>
-
- May 11, 2023
-
-
Braden Shepherdson authored
No `metabase.lib.js` wrapping yet.
-
Ryan Laurie authored
-
metamben authored
* Keep breakout and order-by columns as selectable Fixes #30568. The already selected columns are kept in the lists returned by breakoutable-columns and orderable-column, but they are marked with their position in the breakouts and order-bys list, respectively. * Add test for orderable-columns display-info * Add test for orderByPosition after a roundtrip * Cherry pick order-by cleanup fix * Fix breakoutable and orderable columns after round trip * Declare breakoutPosition in ColumnDisplayInfo * Address review feedback --------- Co-authored-by:
Anton Kulyk <kuliks.anton@gmail.com>
-
Mark Bastian authored
Removes display of underlying type of x-ray card (e.g. model, question) in title. Adds tests.
-
metamben authored
* Support expression options and fields only case and coalasce exprs Fixes #29938. Fixes #29950.
-
Ryan Laurie authored
* don't allow unscoped text selectors in e2e tests * properly scope some recent tests * disable eslint for legacy violations * scope selectors in dashboard drill test
-
metamben authored
* Prevent fields hiding joined fields (#30644) Fixes #29904. * Resolve join columns in the join stages The original implementation got into infinite loops.
-
Natalie authored
-
Anton Kulyk authored
* Add field dimension options types * Add dimension options to sample database mock
-
Alexander Polyankin authored
-
Nemanja Glumac authored
-
Nemanja Glumac authored
* Remove unused props * Remove `index` prop * Do not pass unneeded prop that's not even defined * Remove seven year old TODO * Render `ListItem` with the `li` HTML element
-
Nemanja Glumac authored
-
Anton Kulyk authored
* Add field values to mock sample database * Use correct `has_field_values` in mock sample database * Add `PEOPLE_SOURCE_VALUES` to mock sample database
-
Alexander Polyankin authored
-
Alexander Polyankin authored
-
Alexander Polyankin authored
-
Mahatthana (Kelvin) Nomsawadi authored
-
Tim Macdonald authored
-
Cal Herries authored
* Add schemas driver feature flag * Add sql_jdbc schemas test * Add more general test * Make schemas feature default to true * Implement supports? for other drivers * Add feature to the driver changelog * Remove redundant schemas from h2 * Spelling fix Pre-rebase SHA: 8a2c7e0 or 5686b2f * MySQL does *not* support schemas * Bad rebase * Typo? Co-Authored-By:
Tim Macdonald <tim@metabase.com>
-
Ngoc Khuat authored
* Dashboard, DashboardCard, DashboardSeries to toucan2
-
Ngoc Khuat authored
* revert remove hydrate param values in #22836 * fix tests in ee ns * fix param field tests * Add e2e test * Add e2e tests * Add e2e tests * fix failed test and use t2 instead of db * fix wrong t2 call spec * fix failing tests * Merge branch 'master' into ngoc-revert-remove-hydrate-param-values * Merge branch 'master' into ngoc-revert-remove-hydrate-param-values * apply Tamas's suggestions and unskip e2e tests * delete the e2e tests that AlexP previously added in this PR but brought it into a seperated PR" " * create Field values if not found * add a tests for when field values is not existed, we create them * Merge branch 'master' into ngoc-revert-remove-hydrate-param-values * Fix the e2e test suite for remapping * Fix the e2e test suite for remapping * Fix the e2e test suite for remapping * Fix the e2e test suite for remapping * Fix the e2e test suite for remapping * revert the changes to create full values on demand * remove unused requires * Merge branch 'master' into ngoc-revert-remove-hydrate-param-values * fix tests * Merge branch 'master' into ngoc-revert-remove-hydrate-param-values * Merge branch 'master' into ngoc-revert-remove-hydrate-param-values Co-Authored-By:
Alexander Polyankin <alexander.polyankin@metabase.com>
-
- May 10, 2023
-
-
Natalie authored
-
dpsutton authored
* Alternative GEOJSON.io fix rather than not testing, just accept that we might get an expected location back or we might get nil. This actually represents how the application will behave and will seamlessly work as the service improves or continues throwing errors: ```shell ❯ curl "https://get.geojs.io/v1/ip/geo.json?ip=8.8.8.8,136.49.173.73,185.233.100.23" <html> <head><title>500 Internal Server Error</title></head> <body> <center><h1>500 Internal Server Error</h1></center> <hr><center>openresty</center> </body> </html> ❯ curl "https://get.geojs.io/v1/ip/geo.json?ip=8.8.8.8,136.49.173.73" [{"area_code":"0","organization_name":"GOOGLE","country_code":"US", "country_code3":"USA","continent_code":"NA","ip":"8.8.8.8", "region":"California","latitude":"34.0544","longitude":"-118.2441", "accuracy":5,"timezone":"America\/Los_Angeles","city":"Los Angeles", "organization":"AS15169 GOOGLE","asn":15169,"country":"United States"}, {"area_code":"0","organization_name":"GOOGLE-FIBER","country_code":"US", "country_code3":"USA","continent_code":"NA","ip":"136.49.173.73", "region":"Texas","latitude":"30.2423","longitude":"-97.7672", "accuracy":5,"timezone":"America\/Chicago","city":"Austin", "organization":"AS16591 GOOGLE-FIBER","asn":16591,"country":"United States"}] ``` Changes are basically ```clojure (schema= (s/conditional some? <original-expectation-schema> nil? (s/eq nil) response-from-geojson) ``` * Filter to login error messages ```clojure (into [] (map (fn [[log-level error message]] [log-level (type error) message])) error-messages-captured-in-test) [[:error clojure.lang.ExceptionInfo "Error geocoding IP addresses {:url https://get.geojs.io/v1/ip/geo.json?ip=127.0.0.1}"] [:error clojure.lang.ExceptionInfo "Authentication endpoint error"]] ``` * check timestamps with regex seems like they fixed the service ```shell curl "https://get.geojs.io/v1/ip/geo.json?ip=8.8.8.8,136.49.173.73,185.233.100.23" [{"country":"United States","latitude":"34.0544","longitude":"-118.2441","accuracy":5,"timezone":"America\/Los_Angeles","ip":"8.8.8.8","organization":"AS15169 GOOGLE","country_code3":"USA","asn":15169,"area_code":"0","organization_name":"GOOGLE","country_code":"US","city":"Los Angeles","continent_code":"NA","region":"California"},{"country":"United States","latitude":"30.2423","longitude":"-97.7672","accuracy":5,"timezone":"America\/Chicago","ip":"136.49.173.73","organization":"AS16591 GOOGLE-FIBER","country_code3":"USA","asn":16591,"area_code":"0","organization_name":"GOOGLE-FIBER","country_code":"US","city":"Austin","continent_code":"NA","region":"Texas"},{"country":"France","latitude":"48.8582","longitude":"2.3387","accuracy":500,"timezone":"Europe\/Paris","ip":"185.233.100.23","organization":"AS198985 AQUILENET","country_code3":"FRA","asn":198985,"area_code":"0","organization_name":"AQUILENET","country_code":"FR","continent_code":"EU"}] ``` whereas a few hours ago that returned a 500. And now the timestamps are different ``` ;; from schema "2021-03-18T19:52:41.808482Z" ;; results "2021-03-18T20:52:41.808482+01:00" ``` kinda sick of dealing with this and don't want to deal with a schema that matches "near" to a timestamp. * Restore these i copied changes over from tim's fix and it included these fixes. But now the service is fixed and these are the original values from cam's original fixes
-
Ryan Laurie authored
-
Jesse Devaney authored
-
Ryan Laurie authored
-
Ryan Laurie authored
* update upload button tooltip * add upload tooltip subtitle * update test
-
Nemanja Glumac authored
* Fix wrong operator formatting in filter popover header * Replace custom `field.displayName` implementations * Improve displayName `metabase-lib` method * Evaluate to an actual display name value * Add repro for #27104 * Guard against the missing table
-
Alexander Polyankin authored
-
- May 09, 2023