Skip to content
Snippets Groups Projects
This project is mirrored from https://github.com/metabase/metabase. Pull mirroring updated .
  1. Nov 22, 2024
  2. Nov 21, 2024
  3. Nov 20, 2024
    • Romeo Van Snick's avatar
      Allow searching a single select list picker (#49739) · 79858b9f
      Romeo Van Snick authored
      
      * Allow searching a single select list picker
      
      * Fix unit tests
      
      * Add repro for 49642
      
      * Remove selection when searching
      
      * Rename parameter to value because it does not need to be a prefix
      
      Co-authored-by: default avatarKamil Mielnik <kamil@kamilmielnik.com>
      
      * Simplify loading state for ListField
      
      * Use same DEBOUNCE_FILTER_TIME in ListField
      
      * Don't rely on waitTimeContext in test
      
      * Fix lint
      
      * Add initialization state for filter widget
      
      * Update Loki Snapshots
      
      * Update Loki Snapshots
      
      * Allow picking other values when selecting a single value dropdown value
      
      * Add comment about initialization
      
      * Use setup in test
      
      * Update Loki Snapshots
      
      ---------
      
      Co-authored-by: default avatarKamil Mielnik <kamil@kamilmielnik.com>
      Co-authored-by: default avatarMetabase Automation <github-automation@metabase.com>
      79858b9f
    • Phoomparin Mano's avatar
    • Phoomparin Mano's avatar
      feat(sdk): revamp CreateQuestion and create question behaviour (#50088) · 5fb95767
      Phoomparin Mano authored
      * revamp the create question component
      
      * docs updates
      
      * add stories and update styles
      
      * prevent switching to visualization when it is not ready
      
      * ensure CreateQuestion works in flex parents
      
      * add basic e2e tests for CreateQuestion
      
      * ensure that switching between editor and visualization works
      
      * use the save button for the disabled state
      
      * update the question title when the question is saved
      
      * implement updating questions in place
      
      * rename replaceQuestion
      
      * make CreateQuestion props all optional
      
      * add background color to modal header
      
      * hide the save button if question is dirty
      
      * hide save button instead of disabling per design feedback
      
      * update e2e tests
      
      * revert e2e helpers
      
      * add title update assertions
      
      * whitespace changes
      5fb95767
    • Ngoc Khuat's avatar
    • Kamil Mielnik's avatar
      Fix - Notebook expression editor autocomplete issues (#49949) · 12f85cbb
      Kamil Mielnik authored
      * Allow navigating with arrow up/down between lines in expression editor
      
      * Do not hijack up/down arrow keys for autocomplete suggestions navigation when there's only 1 suggestion
      
      * Add explanatory comments
      
      * Convert tokenizer to TS
      
      * Add a test
      
      * Format code
      
      * Add basic test case
      
      * Fix a deprecation
      
      * Handle incomplete bracket identifier tokens
      
      * Shorten code
      
      * Add comment
      
      * Fix suggestion not always getting selected
      
      * Fix computation
      
      * Fix types
      
      * Update comment
      
      * Add repros for #49882
      
      * Add repro
      
      * Fix comment
      
      * Deprecate a HOC
      
      * Update assertion
      
      * Remove _.compose and default export
      
      * Fix types in ExplicitSize
      
      * Fix typing in ExpressionEditorTextfield
      
      * Fix types
      
      * Fix error handling
      
      * Fix showing errors when expression is invalid
      
      * Do not submit null expressions
      
      * Fix typing
      
      * Update assertion
      
      * Fix flaky test
      
      * Fix filter
      
      * Decrease flakiness
      
      * Fix crashing
      
      * Add comment
      
      * Fix incomplete bracket idenfitiers
      
      * Fix type
      
      * Improve typing
      
      * Add an explanatory comment
      12f85cbb
    • Mahatthana (Kelvin) Nomsawadi's avatar
      Quarantine the flaky assertions (#50198) · 9ed2c3a2
      Mahatthana (Kelvin) Nomsawadi authored
      * Refactor: Speed up E2E tests by not relying on UI
      
      * Quarantine flaky E2E assertions
      9ed2c3a2
  4. Nov 19, 2024
  5. Nov 18, 2024
    • Nick Fitzpatrick's avatar
      search result order (#50048) · 21ae344b
      Nick Fitzpatrick authored
      21ae344b
    • Kamil Mielnik's avatar
      Fix flaky x-rays test (#50131) · d291c150
      Kamil Mielnik authored
      * Fix flaky test
      
      * Add extra assertion
      
      * Reset hover state
      
      * Change order of assertions
      d291c150
    • lbrdnk's avatar
      Add temporal unit to display name (#48946) · 5203a15c
      lbrdnk authored
      * Add temporal unit to display name
      
      * Remove temporal unit adjustments from FE
      
      * formatColumn to displayNameForColumn
      
      * Factor out getFriendlyName
      
      * Adjust BE tests
      
      * Add require
      
      * Adjust FE unit tests
      
      * Adjust temporal-unit-parameters.cy.spec.js
      
      * Adjust dashboard-reproductions.cy.spec
      
      * Convert formatColumn occurrence
      
      * Adjust legacy display name logic generation
      
      After `getFriendlyName` was removed from FE, capitalization
      of display name should happen here. Logic is similar, should
      be adjusted prior merging the PR.
      
      * Adjust embedding-linked-filters.cy.spec.js
      
      * Adjust joins-reproductions.cy.spec.js
      
      * Adjust metrics-editing.cy.spec.js
      
      * Adjust timelines-question.cy.spec.js
      
      * Adjust offset.cy.spec.ts
      
      * TMP Avoid duplication of temporal unit in display name in MLv2 code
      
      * Probably FE fix: add default temporal bucket only if not present
      
      * WIP Add ends-with-temporal-unit? function
      
      Current strategy is to add temporal units to display names if same is not already present
      
      * Adjust display-name-method :metadata/column
      
      Append TU to display name only if different to what's already present
      
      * Adjust FE: show temporal unit that's present on field instead of default on hover on QueryColumnPicker element
      
      * Return also temporal-unit from visible-columns
      
      * Enable tempora-unit to be returned from previous-stage-or-source-visible-columns
      
      * WIP Adjust multi breakout multi stage e2e multiple-column-breakouts.cy.spec
      
      * Adjust multiple-column-breakouts.cy.spec.ts
      
      * It was not a bug.
      
      * Revert changes in _BaseBucketPickerPopover
      
      * Restore temporal-unit for next stages removal in stage.cljc
      
      * Add inherent-temporal-unit
      
      * Adjust display name temporal bucket logic
      
      * Rename to inherited-temporal-unit
      
      * Shutdown too broad display name capitalization
      
      * Eastwood
      
      * Adjust pivot-rows-order-test
      
      * Adjust pivot-non-numeric-values-in-aggregations
      
      * Adjust inherent to inherited
      
      * Rename to inherited
      
      * Add inherent-temporal-unit only to returned-columns
      
      * Update e2e/test/scenarios/dashboard-filters/temporal-unit-parameters.cy.spec.js
      
      * Update e2e/test/scenarios/question/multiple-column-breakouts.cy.spec.ts
      
      * Update consistent-date-formatting-test
      
      * Update dashboard/x-rays.cy.spec.js
      
      * Update waterfall.cy.spec.js
      
      * Return month default for default inherited-temporal-unit
      
      * Update reproductions-3.cy.spec.js
      
      * Update downloads/downloads.cy.spec.js
      
      * Update legend.cy.spec.js
      
      * Update pie_chart.cy.spec.js
      
      * Update click-behavior.cy.spec.js
      
      * Update embedding-linked-filters.cy.spec.js
      
      * Return QueryColumnPicker's condition to original state
      
      * Remove reundant export
      
      * Update e2e/test/scenarios/question/multiple-column-breakouts.cy.spec.ts
      
      * Comment
      
      * Update nested.cy.spec.js
      
      * Update e2e/test/scenarios/sharing/sharing-reproductions.cy.spec.js
      
      * Adjust to TS types
      
      * Adjust for cljfmt
      
      * Capitalize
      
      * Add propagation tests
      
      * Add inherited-temporal-unit propagation for expressions
      
      * Add default inherited-temporal-unit test cases
      
      * Add qp temporal-bucket-test ns
      
      * Simplify default unit logic
      
      * Comment
      
      * Use NULL_DISPLAY_VALUE
      
      * Comment
      
      * Comment
      
      * Comment
      
      * Comment
      
      * Comment
      
      * Comment
      
      * Remove redundant code (for now)
      
      * Update comments
      
      * Comment
      
      * Restore way too daring simplification
      
      * cljfmt
      
      * Adjust capitalize
      
      * Update comment
      
      * Update test
      
      * Address remarks in tests
      
      * Update test
      
      * Address remarks
      
      * typo
      5203a15c
    • Uladzimir Havenchyk's avatar
      fix: respect click behavior on public dashboards (#50067) · 552cf921
      Uladzimir Havenchyk authored
      * fix: pass noop as navigateToNewCardFromDashboard from public or embedded dashboard
      
      * add e2e test
      
      * rework a fix
      552cf921
    • Romeo Van Snick's avatar
      Fix button group breaking (#50064) · c8aa0a54
      Romeo Van Snick authored
      
      * Use Flex instead of Group for items in button
      
      * Allow column picker to stretch to match the height of the element
      
      * Fix height of data picker buttons
      
      * Make sure the whole button is clickable
      
      * Add e2e test for metabase/metabase#50038
      
      * Use correct id for issue in e2e test
      
      Co-authored-by: default avatarKamil Mielnik <kamil@kamilmielnik.com>
      
      ---------
      
      Co-authored-by: default avatarKamil Mielnik <kamil@kamilmielnik.com>
      c8aa0a54
    • Kamil Mielnik's avatar
      Fix flaky test (#50077) · 17887dcd
      Kamil Mielnik authored
      17887dcd
    • Nicolò Pretto's avatar
      fix(49537): remove position relative from echarts tooltip container (#49964) · 7951b711
      Nicolò Pretto authored
      
      * attempt to fix 49537 by removing position relative from the tooltip
      container
      
      * Make echart tooltip container cover all iframe area
      
      * Update the E2E test
      
      * Update loki snapshots from CI
      
      * Re-enable echarts tooltip assertion.
      
      I couldn't find a better way to fail these assertions on the previous
      version of the code, but these assertions are better than nothing.
      
      ---------
      
      Co-authored-by: default avatarMahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>
      7951b711
  6. Nov 15, 2024
  7. Nov 14, 2024
  8. Nov 13, 2024
  9. Nov 12, 2024
Loading