Co-authored-by: Oisin Coveney <oisin@metabase.com>

Co-authored-by: Oisin Coveney <oisin@metabase.com>

Co-authored-by: Oleksandr Yakushev <alex@bytopia.org>

Co-authored-by: Oleksandr Yakushev <alex@bytopia.org>

* Add missing -test suffix to zoom-in-timeseries test name

* Add missing -test suffix to underlying-records test name

* Fix distribution drill thru for multi-stage queries

Ensure the distribution drill thru is not returned by available-drill-thrus when a user clicks on an aggregation or
breakout column, even if there are additional query stages after the last aggregation / breakout stage.

Co-authored-by: appleby <86076+appleby@users.noreply.github.com>

Co-authored-by: Aleksandr Lesnenko <alxnddr@users.noreply.github.com>

Don't compute a different hash key if they only differ on the order of values in a parameter (#51186) (#51306)

Don't compute a different hash key if they only differ on the order of values in a parameter

Co-authored-by: Nathan Voxland <nathan@metabase.com>

Some funcs were added to lib.drill-thru.common for the recent fix to the zoom-in.timeseries drill. Update the
underlying-records drill to use the common lib code rather than duplicating it.

Co-authored-by: appleby <86076+appleby@users.noreply.github.com>

Co-authored-by: Ryan Laurie <30528226+iethree@users.noreply.github.com>

It looks like snowflake introduced some very noisy logging in their
code.

https://github.com/snowflakedb/snowflake-jdbc/commit/9b7c9eb3e02c93cf7788c9866e1a483fe978e1be#diff-7859cac7243b3b9668df0770c96e264a1be2be06eefbbf7bba63904104028495L743



This should reduce the noise and allow our code to handle and log the
exceptions.

Co-authored-by: Case Nelson <case@metabase.com>

We changed the time_history icon in this context to a revert icon in #46050 but never updated this test. This PR fixes that.

Co-authored-by: Raphael Krut-Landau <raphael.kl@gmail.com>

* save only grid on fixed width dashboards

* split export pdf into pages

* trim dashboard export pdf into pages

* specs

* add padding to pdf pages

* dark mode

Co-authored-by: Aleksandr Lesnenko <alxnddr@users.noreply.github.com>

Co-authored-by: Chris Truter <crisptrutski@users.noreply.github.com>

Co-authored-by: Oisin Coveney <oisin@metabase.com>

* Fix #51020

* Add a repro for #51020

* Improve test name

* Update comment and naming

* Improve diff

* Add repro for when primary key is not called 'id'

* Update repro

* Drop prefetchesInitialValues & getValuesForUpdate

It's unclear what the real intention behind these two was, but it's clear that it made two differences:
(1) It decreased the probability of overwriting someone else's changes (if someone else saved them after we opened the form)
(2) It decreases the payload sent over the wire

(1) We can ignore this, as this problem is prevalent everywhere in our app
(2) This is negligible

Dropping these is simpler than enhancing getValuesForUpdate with extra logic for primary keys (and who knows what else)

* Break beforeEach into 2

* Remove no longer valid test

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

should be reverted but we want a 52 release

context:
https://github.com/metabase/metabase/actions/runs/12307551790/job/34351648561

* guard against a stage number in matching

old:

`{"category":["dimension",["field",5,{"base-type":"type/Integer"}]]}`

new:

`{"state":["dimension",["field",48,{"base-type":"type/Text"}],{"stage-number":0}]}`

(comment
  (let [login-attributes {"state" "CA"}
        field-ids #{55 46 54 48 50 56 51 47 57 45 53 52 49}
        attribute_remappings {"state" [:dimension
                                       [:field
                                        48
                                        {:base-type :type/Text}]
                                        ;; this stage-number is optional. Newer versions have it since https://github.com/metabase/metabase/issues/46914
                                       {:stage-number 0}]}]
    (into {} (for [[k v] attribute_remappings
                   ;; get attribute that map to fields of the same table
                   :when (contains? field-ids
                                    (lib.util.match/match-one v [:dimension [:field field-id _]] field-id))]
               {k (get login-attributes k)})))
  )

* formatting fix

Co-authored-by: Raphael Krut-Landau <raphael.kl@gmail.com>

Co-authored-by: Ryan Laurie <30528226+iethree@users.noreply.github.com>

* skip tracking non react-sdk x-metabase-clients

- Do not do promethus tracking for sdk.
- tracking ignores x-metabase-client headers when they're not == "embedding-sdk-react"

* cleanup indentation

* add prometheus tracking counters for iframe

* remedy 24h query execution inaccuracy

Co-authored-by: bryan <bryan.maass@gmail.com>

Allow config files to have literal values enclosed in {{{ ... }}}

Co-authored-by: Nathan Voxland <nathan@metabase.com>

Co-authored-by: Phoomparin Mano <poom@metabase.com>
Co-authored-by: Denis Berezin <denis.berezin@metabase.com>

Tiny typo

Co-authored-by: Hervé Labas <431257+thebiglabasky@users.noreply.github.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

* add bug report to command palette and profile menu

add bug report to command palette and profile menu

* refactor and update diagnostic window modal

* add analytics to track bug reporter modal activity

* add file id to to the upload-file method

* add the ability to collect browser information for diagnostic window

* implement feedback from PR comments

* Update frontend/src/metabase/components/ErrorPages/ErrorDiagnosticModal.tsx



* implement feedback from PR comments #2

* rename "file a bug" to "report an issue"

* resolve conflict

* fix lint issues

* fix bad merge conflict resolution

* fix failing onboarding tests

---------

Co-authored-by: Filip Hric <filip@filiphric.sk>
Co-authored-by: Raphael Krut-Landau <raphael.kl@gmail.com>

Co-authored-by: Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>

Co-authored-by: Chris Truter <crisptrutski@users.noreply.github.com>

Co-authored-by: Chris Truter <crisptrutski@users.noreply.github.com>

Co-authored-by: Chris Truter <crisptrutski@users.noreply.github.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

* Prevent newly opened items from expanding outside viewport

* Expand the unti test coverage

* Use the existing helper

* Change scroll view to center

This works better because it ensures that the top bar doesn't cover the element.

* Update frontend/src/metabase/home/components/Onboarding/Onboarding.tsx



* Extract the `stopVideo` function

* Fix the type name

* Set the top scroll margin on the `main` element

---------

Co-authored-by: Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>
Co-authored-by: Romeo Van Snick <romeo@romeovansnick.be>

* wip: better documentation for developers that need to run the sdk
locally

* morning fixes

* remove developers-guide/sdk in favor of dev.md in the sdk folder

* add note about using the sdk with link to the docs

* fix mising word

* addressed feedback

* added notes about the envs for sdk e2e test

Co-authored-by: Nicolò Pretto <info@npretto.com>

* Force nanoid onto 3.3.8

* Force path-to-regexp to 0.1.12

Co-authored-by: Romeo Van Snick <romeo@romeovansnick.be>

Co-authored-by: Oisin Coveney <oisin@metabase.com>

Co-authored-by: Phoomparin Mano <poom@metabase.com>

* make release log updates smarter

* fix a bunch of stuff after testing

Co-authored-by: Ryan Laurie <30528226+iethree@users.noreply.github.com>

* Add v0.51.8 to the list of releases

* ordering

---------

Co-authored-by: Metabase bot <metabase-bot@metabase.com>
Co-authored-by: jeff bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Ngoc Khuat <qn.khuat@gmail.com>

* feat(sdk): use public-facing question type in event handlers (#50867)

* introduce public metabase question type

* revise internal question type

* update stories

* use public question type in dashboard card plugins

* add tests for transforming question properties

* add tests for question description

* re-export the MetabaseQuestion type

* fix lint imports

* change publicQuestion to sdkQuestion

* name cannot be null if the card is present

* reduce collection browser flake

---------

Co-authored-by: Phoomparin Mano <poom@metabase.com>
Co-authored-by: Phoomparin Mano <phoomparin@gmail.com>

Co-authored-by: Alexander Solovyov <alexander@solovyov.net>