This project is mirrored from https://github.com/metabase/metabase.
Pull mirroring updated .
- Oct 17, 2023
-
-
Nemanja Glumac authored
* Conditionally start Docker containers for E2E tests * Check out before initializing Docker containers * Use official action to log in to DockerHub * Evaluate conditional expressions * Try shell conditionals * Fix errors due to unset vars * Run either Mongo or Pg+MySQL
-
Nemanja Glumac authored
* Remove `branch` from version.properties file * Remove branch from logs * Remove `branch` from `application-name` The `application-name` string is needed for Google drivers. See: https://github.com/metabase/metabase/issues/2627
-
Case Nelson authored
https://www.notion.so/metabase/Filter-clause-text-descriptions-556639c83bd24e3f9a306b68d36e8a1f Co-authored-by:
metamben <103100869+metamben@users.noreply.github.com> Co-authored-by:
Anton Kulyk <kuliks.anton@gmail.com>
-
lbrdnk authored
* Update migration and add test Sets semantic_type to null for metabase_field if value was type/Number, to avoid log warnings. * Remove test for trivial migration as suggested As migration tests are costly and change is trvial, it is better to test small migration as one in question manually. Hence removing the test.
-
metamben authored
Enable column-filter, disable distribution and summarize-column drills for aggregated columns (#34681)
-
metamben authored
-
Cam Saul authored
-
Nicolò Pretto authored
-
Kamil Mielnik authored
* Update isNavigationAllowed logic to prevent new model creation * Add simple model creation test * Remove identifier * Update comment * Add a test case for saving a new model * Avoid using fetchMock directly in QueryBuilder unit tests * Remove setupCollectionsEndpointsByIds in favor of setupCollectionByIdEndpoint * Add native question test base * Update isNavigationAllowed tests * Make isNavigationAllowed tests pass * Update shouldShowUnsavedChangesWarningForSqlQuery logic to account for new questions * Add a test case to run new native question * Update mock to include individual collection GET endpoint * Add DataSourceSelectors to MockNativeQueryEditor * Update test beforeunload event to reflect new behavior * Use setupCollectionByIdEndpoint instead of modifying setupCollectionsEndpoints * Update failing test * Use serializeCardForUrl * Add a test for native query editing * Do not trigger warning when leaving empty question * Add beforeunload tests for creating new questions * Remove redundant div * Make assertions consistent * Use single resetAllMocks() * Fix post-rebase conflict * Work around Cypress not handling beforeunload-related prompts * Our app registers beforeunload event listener e.g. when editing a native SQL question. * Cypress does not automatically close the browser prompt and does not allow manually * interacting with it (unlike with window.confirm). The test will hang forever with * the prompt displayed and will eventually time out. We need to work around this by * monkey-patching window.addEventListener to ignore beforeunload event handlers. * * @see https://github.com/cypress-io/cypress/issues/2118
-
Nemanja Glumac authored
* Make `Chromatic` workflow 3rd-party friendly * Remove unused output * Unify the use of `GITHUB_TOKEN` * Update label check * Fix wrong token reference [ci skip]
-
Nemanja Glumac authored
This was causing scheduled runs to fail due to a missing closing quotes in a string. [ci skip]
-
- Oct 16, 2023
-
-
Jeff Bruemmer authored
-
Uladzimir Havenchyk authored
-
Alexander Solovyov authored
fixes #34615
-
Cal Herries authored
-
Ryan Laurie authored
-
Cal Herries authored
-
Kamil Mielnik authored
* Update isNavigationAllowed logic to prevent new model creation * Rename tests * Add routes for model creation * Allow null card in setup * Add test case for creating models in isNavigationAllowed * Add simple model creation test * Add workaround for metabase#34523 * Remove "Discard changes?" confirmation modal when creating a model - use a single "Cancel" button - remove custom cancel edit warning * Update e2e test * Remove redundant function bodies * Bring back unsaved changes warning on cancel click when editing a model * Add beforeunload event test for creating models * Fix unit test * Fix unit test and isNavigationAllowed * Use regexp * Add notebook prefix * Drop mock prefix * Use it.each * Use it.each * Use it.each * Use it.each * Use it.each * Use it.each * Refactor isNavigationAllowed * Remove identifier * Update comment * Add a test for the cancel button * Rename /home route to / so that redirection after cancelling creating a model works * Add a test case for saving a new model * Avoid using fetchMock directly in QueryBuilder unit tests * Remove setupCollectionsEndpointsByIds in favor of setupCollectionByIdEndpoint * Remove default value for card parameter
-
lbrdnk authored
* Update join condition transformation for mongo LHS of let in lookup must contain identifier with no dots. This commit updates identifier generation to conform that. * Add review suggestions
-
Cal Herries authored
-
Nemanja Glumac authored
* Move browser-related Cypress prep steps to a composite action * Fix browser paths
-
Cal Herries authored
-
Cal Herries authored
-
Lena authored
Remove the emojis from the team labels. With the emojis, the "auto-add to project" github automated workflow doesn't work, and we rely on it for the #bug-triage channel on slack. After several attempts of escaping the emojis, they've been removed from the labels, and now have to be removed here as well for the auto-assignment of PRs to continue working.
-
Cal Herries authored
-
Alexander Solovyov authored
-
Nemanja Glumac authored
* Bump Postgres in drivers tests to 12 * Update env vars * Authenticate with password * user name with prefix pg_ is not allowed --------- Co-authored-by:
Ngoc Khuat <qn.khuat@gmail.com>
-
- Oct 14, 2023
-
-
Tim Macdonald authored
-
- Oct 13, 2023
-
-
Jerry Huang authored
-
Denis Berezin authored
* Add Sort drill * Fix unit tests * Add more unit/integration tests * Add more unit/integration tests * Add more unit/integration tests
-
Anton Kulyk authored
* Collapse temporal units list * Extract `MoreButton` component * Simplify conditions * Make `isInitiallyExpanded` a bit more clear * Use `INITIALLY_VISIBLE_ITEMS_COUNT` in tests * Fix typos * Remove comment * Update popover size and spacing * Remove outdated test * Update E2E tests * Add hover state to the "More" button * Disable hover transition
-
Anton Kulyk authored
-
Cam Saul authored
-
John Swanson authored
This namespace is deprecated and replaced with `java-time.api`. https://github.com/metabase/metabase/issues/34506
-
Ngoc Khuat authored
-
Cal Herries authored
-
Anton Kulyk authored
-
Alexander Solovyov authored
fixes #31153 Co-authored-by:
Mayank Agarwal <mayankagarwal44442@gmail.com>
-
Cal Herries authored
FE: Do not show linked filter options if the parameter has values_source_type of "static-list" or "card" (#34438)
-
Oisin Coveney authored
-