* Replace `isomorphic-fetch` with plain `XMLHttpRequest`

* Remove `isomorphic-fetch`

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

* untie getColumnKey function from unimportable in static viz environment code

* review

* spec

* validate dashcard-id in addition to card-id when running a query for a dashcard

* basic test

* update tests with dan's suggestion

* correctness fix and dashcard series tests

* set initial sync status of crufty tables to complete automatically

* fix test

* remove actionClickDrill and stop using click behavior for actions

* close modal on success

* fix type

Repro #24660: Saved Questions data selector lists all questions for collections with the same name (#25919)

* Move subset of parameter-values utils to metabase-lib

* Move parameter mbql utils to metabase-lib

* Create some small target utils

* Move template tag utils to metabase-lib

* Move target/field logic to metabase-lib

* Remove eslint-enable

* Unrelated fix: app DB data source should assume default host/port for MySQL/Postgres

* Remove stray println

enviornment -> environment

* Add CLI SerDes command for seeding entity IDs (#25655)

* Remove stray comment

* Fix Kondo error

* Fix Eastwood error

* Update identity-hash-fields to multimethod for ModelAction

* Fix identity-hash-fields for Dimension and DashboardCardSeries

* Return models used in scaffolding in GET /app/:id
* Return models added to an app page
* Return models referenced by queries on app pages

* [Apps] Model Actions

WIP Adding migration and endpoints for model_action.

* WIP model action execution

Changing the execution endpoints again, still working on implicit action
execution tests. Need to also test the parameters fetch.

* Re-add execution route for dashcard action_id until Front End catches up

* Add name to model_action GET

* Hydrate model-action on dashcards

* Rename and combine hydration of model_action to action

* Implicit action execution support. Use slug columns as parameter ids

* Go through model-action when using GET /action

* FE integration fixes

* Move action execution to use new parameter shape

Now, parameters should be a map of parameter-id to value
Parameter-id should be mapped on the FE. So the action parameter-id not the unmapped
dashboard parameter-id.

* Bring ModelAction into copy code

* Generate target of implicit actions to match http actions

* Update action test to place action on model

* Add unique constraint to model_action for (card_id, slug)

* Add missing require

* model_action.slug needs to be varchar for mysql constraint

* Fix test from unique constraint

* Consistent ordering for tests

* Addressing Code Reviews and Scaffold changes

* Add type: implicit to implicit model actions

* Update src/metabase/api/model_action.clj

Co-authored-by: metamben <103100869+metamben@users.noreply.github.com>

* Allow changing card_id with PUT to dashcard for model-actions

* Update tests after merge

* Fix implicit delete and add better test coverage

* Addressing review changes

Add PUT test
Add schema for dashboard-id and dashcard-id params
Fix select ordering in test

* save actions in models

* list model actions on detail page (#25648)

* Link Dashcards to model-based actions (#25770)

* Link dashcards to model-based actions

* address review comments

* Data apps model actions implicit creator (#25807)

* Link dashcards to model-based actions

* address review comments

* Link dashcards to model-based actions

* Allow implicit actions to be used like explicit ones

* address review comments

* update action types

* add action sidebar

* remove action selection from click behavior sidebar

* rework action picker

* add shortcut to create an action for a model

* update tests

* address review comments

Co-authored-by: Case Nelson <case@metabase.com>
Co-authored-by: metamben <103100869+metamben@users.noreply.github.com>
Co-authored-by: Anton Kulyk <kuliks.anton@gmail.com>

* Initial Attempt

* Adding Divider

* PR Feedback

* [Apps] Small scaffolding bug fixes

Detail Object should use dimension target
List link should use pk-field-name

* Fix test - ordered_cards behaving oddly

* source id should be pk-field name

* Refactoring ChartSettingFieldsPartition DnD logic

* small adjustments

* removing unneeded state

* PR Feedback, clean up dragEnd Logic

* Updating cypress test

* More PR Feedback

These changes resulted from the effort to serialize and deserialize some
collections from our internal Stats instance, which did not
(de)serialize correctly out of the box.

The main improvement is more robust "escape" analysis - making sure that
all dashboards, cards, models, etc. required by the curated collections
actually belong to those curated collections. (Many cards on Stats were
actually in people's personal collections and publicly readable - that
doesn't work with the curated git workflow.)

This adds some new tests, trying to capture some complex MBQL queries
and make sure they're correctly (de)serialized.