Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

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

* normalize icon sizes and spacing

* clean up some types

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

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

Co-authored-by: Cam Saul <1455846+camsaul@users.noreply.github.com>

* Properly connect actions to search entity

* Add e2e test

* adding hover states, switching icons for buttons

* ChartSettingFieldsPicker unit tests

* fixing background hover state

* using userEvent.click instead of fireEvent

* adjusting e2e tests

Co-authored-by: Nick Fitzpatrick <nick@metabase.com>

Co-authored-by: Natalie <nat@metabase.com>

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

* Bind *card-id* when executing a dashboard subscription to fix perm checks (#28961)

* bind *card-id* when executing a dashboard subscription so that perm checks work properly

* refactor & address test comment

* fix spacing

* unskip repro that this also fixes

* fix lint

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

* Fix dashboard filter values coercsion for actions

* update custom action e2e test to cover parameters

---------

Co-authored-by: Anton Kulyk <kuliks.anton@gmail.com>
Co-authored-by: Ryan Laurie <iethree@gmail.com>

* Remove redundant `isSettings` prop

* Extract `FormFieldEditorDragContainer`

* Extract `FormFieldEditor` component

* Update field editor UI

* Disable validation in parameters editor

* Fix crash on adding a new parameter

* Add new tests and fix existing

* Remove redundant `overflow: hidden`

* Use shorter flex properties

* Remove redundant `key` prop

* Increase icon size

Co-authored-by: Anton Kulyk <kuliks.anton@gmail.com>

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

* Centering tab labels

* adding unit tests

* Switching to userEvent

Co-authored-by: Nick Fitzpatrick <nick@metabase.com>

* fix the flaky test for Bigquery

* adds a limit to make sure we don't have a big charge on CI

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

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

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

Co-authored-by: Noah Moss <32746338+noahmoss@users.noreply.github.com>

* Hide dashcard parameter mappings for action dashboard cards (#28813)

* hide action parameter mappings on dashcards

* test DashCardCardParameterMapper

* use a better metadata mock

* don't backport test

---------

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

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

Co-authored-by: Natalie <nat@metabase.com>

* WIP action data type tests

* DRY up the actions tests

* lots of type tests

* make the tests work

* reorganize tests

* WIP action data type tests

* DRY up the actions tests

* lots of type tests

* make the tests work

* reorganize tests

* reduce some flakes

* better time formatting in action forms

* sort out all the timezones 



* update after rebase

* rebase disaster

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

22788 fix custom column filter lose their settings when editing dashboards a second time (#28903) (#28913)

* Refactor: Simplify code

* Fix custom field parameter lose their settings

Ref: https://github.com/metabase/metabase/issues/22788 this fixes the
issue where custom field parameter seems to be unselected when editing
dashboard a second time. The reason is because how current Metabase lib
is handling expression's MBQL. In FE, we uses `["expression", "name",
null] while in BE, we will normalize it to `["expression" "name"]`.

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

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

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

Previously, JS files isn't picked up by VS Code, so they'll have limited
intellisence. Also, Jest and Cypress has conflicting names e.g.
`describe` or `expect`. So, this commit attempt to resolve that as well.

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

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

Co-authored-by: Anton Kulyk <kuliks.anton@gmail.com>

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

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

* Ignore JS tests when doing CodeQL scans

* Do not run CodeQL scans unless needed

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

* Use GitHub basic runners for `druid` and `sparksql` tests

* Reduce the `drivers` timeout to 45 mintues

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

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

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

Co-authored-by: Gustavo Saiani <gus@metabase.com>

* initial implementation of query_execution retention policy

* fix logic and add tests for setting

* refactor and more tests

* set default on cloud to 0

* try to fix test by loading ee ns if available

* address feedback and small refactors

* interface file for truncate-audit-log

* extract common definitions into an interface to avoid cyclic dependency

* fix indentation

Co-authored-by: Noah Moss <32746338+noahmoss@users.noreply.github.com>

* sort stacked tooltip rows

* update separator color

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