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

* List all actions

previously `api/action` required `?model-id=<model-id>`. This caused the
FE to DOS the backend looping over each action. On stats I found 350 or
so sequential requests, with app-db saturation leading to requests
taking from 400ms near the beginning to 7seconds near the end.

This makes `model-id` option and now returns actions on all models the
user can see.

```
❯ http GET "localhost:3000/api/action" Cookie:$SESSION -pb
[
    {
        "archived": false,
        "creator": {...},
        "creator_id": 1,
        "database_id": 3,
        "dataset_query": {...},
        "id": 1,
        "model_id": 1,
        "name": "source = foo",
        "parameter_mappings": null,
        "parameters": [...],
        ...
    }
    ...
]
```

* update frontend for action picker to support single action list endpoint

* list actions tests

* update actions e2e tests

* Return empty vector if no results

Running tests locally might return some extra actions not asserted on in
the tests. And there you might expect a 200. But CI runs with an empty
database and returns no response so you get a 204. Quite annoying and
there's no good way to expect a "2xx" status code at the moment. So just
return the empty vector so it's always a 200 code.

* update actions endpoint unit test mocks

* Simplify logic a little

* docstring tweak

* schema with better error message

* update mocks to accept optional modelId

* simplify sorting and grouping

* make sort case-insensitive

* simplify action api test mocks

---------

Co-authored-by: dpsutton <dan@dpsutton.com>
Co-authored-by: Ryan Laurie <iethree@gmail.com>

* Remove personal collections from collection detail page

https://github.com/metabase/metabase/pull/28023

 Added "sub collections"
to the collection detail page. Makes sense. The problem is that personal
collections started appearing in this view. We have traditionally
treated personal collections as siblings to the "Our Analytics" root of
collections. So this was an impedence mismatch.

This PR just removes all personal collections from the items returned
from `"api/collection/root/items"`. The personal collection is still
visible in the sidebar (returned from
`api/collection/tree?tree=true&exclude-other-user-collections=true&exclude-archived=true`
. Furthermore, all personal collections are still visible from the
"three dots" menu on the sidebar.

* poke ci

* remove last personal collections from tests

* e2e tests

don't include a personal collection in the details page.

* off by one from removing personal collections

* Can't select personal collections if they aren't there

Co-authored-by: dpsutton <dan@dpsutton.com>

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

* pulling out column title logic and sharing with settings

* Pivot Table Custom Column Names

* removing cyclic dependency

* adjusting tests

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

* Exclude non-required PKs from implicit create actions

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

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

Co-authored-by: Ngoc Khuat <qn.khuat@gmail.com>
Co-authored-by: Cal Herries <39073188+calherries@users.noreply.github.com>

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

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

* Adding classname prop to ObjectDetail

* making className optional

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

* Don't ignore manually entered parameters on clean

* Validate options for numeric fields with dropdown input

* Clear `valueOptions` when switching field and input types

* Remove sample options for inputs with options

* Rename `getValidationError`

* Bring sample options back

* Minor rename for clarity

* Fix `InputOptionsMap` type

* Keep `inputType` on field type change if possible

* Extract `FieldValueOptions` type

* Use less aggressive value options clean logic

* Simplify tests

* Test value options reset for option-less field types

* Fix `OptionEditor` value gets out of sync

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

Occasionally in CI we get

```
“Syntax error compiling at (release/version_info_test.clj:46:26).\nNo such var: json/read-json\n”,
```

from `release.version-info-test`.
This namespace requires `[clojure.data.json :as json]` and uses the
function `json/read-json` which is deprecated since version 2.x.

The way that deprecated function is defined in version 2.0.2 is

```clojure
(ns clojure.data.json)
...
(load "json_compat_0_1")
```

and in `clojure/data/json_compat_0_1.clj`:

```clojure
(in-ns 'clojure.data.json)

(defn read-json
  "DEPRECATED; replaced by read-str.
  ...)
```

And I guess the way that hawk requires some namespaces we can get into
this half loaded, half not finished state and the var is not (yet)
defined.

So we can just put aside this nonsense and use a non-deprecated var that
isn't defined with a `load` statement but in the regular namespace we
require.

Co-authored-by: dpsutton <dan@dpsutton.com>

* improved datetime handling

*  more tests 



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

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

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

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>