* Fix missing type property on a card breaks frontend UI

* Add E2E test

* MISC improvements

* Update test/metabase/models/secret_test.clj

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

* Update .clj-kondo/hooks/clojure/test.clj

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

* Update .clj-kondo/hooks/clojure/test.clj

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

---------

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

* Improves consistency of substring in MBQL

Standard SQL SUBSTRING() uses a 1-based start index, with various drivers
handling a 0 start index in different ways. To make the behavior of substring
consistent across drivers, we will always convert a 0 start index in MBQL
to a 1 in SQL. The ->honeysql implementation of substring has been generalized
and drivers can now override just the name of the substring function, or the
entire expression if necessary. SQL Server now calls the parent implementation.

Resolves #12445

* Fixes substring from 0 in Mongo

* Adds arg to substring parent call

* Fix docs and helper text for substring()

* Adds schema to require positive integer index for substring

* WIP, adds frontend validation for substring index arg

* Adds frontend test for substring arg validation

* Updates substring test to expect exception on index 0

* Update src/metabase/driver/sql/query_processor.clj

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

* Use s/if instead of s/conditional

* Fix substring call in mysql substring spec

* Reverts driver and sql.qp changes for substring index

* Reverts changes to dispatch-by-clause-name-or-class

* Fixes typeinferencer test substring index

* Adds test for canonicalizing substring

* Fixes substring canonicalization with extra args

* Updates string extracts test to account for substring canonicalization

* Rewrites resolver validation application

* Adds test case for frontend expression validation errors

* Fixes frontend compiler spec for new substring validation

---------

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

* Custom migrations

Current syntax:

specify the migration with
```
  - changeSet:
      id: v46.00-080
      author: dpsutton
      comment: Uppercases all Card names
      changes:
        - customChange:
            class: "metabase.db.custom_migrations.ReversibleUppercaseCards"
```

and in the new namespace metabase.db.custom-migrations:

```clojure
(defmigration UppercaseCards
  (db/execute! {:update :report_card
                :set    {:name :%upper.name}}))

(def-reversible-migration ReversibleUppercaseCards
  (db/execute! {:update :report_card
                :set    {:name :%upper.name}})
  (db/execute! {:update :report_card
                :set    {:name :%lower.name}}))
```

* Use db provided by liquibase

* edit docstring

* set *warn-on-reflection* to fix lint error & rebase

---------

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

* Fix #15324

* Update tests

* Misc QP test improvements

* Add test

* Expand the glob to include TS(X) files

* Use console error

* Re-order imports

* Print the missing files in red

* Fix glob pattern

* Add script to pre-commit hooks

* Remove the check from CI

Resolves #27885

This'll be revisited as part of v47

[Fixes #28425]

* Handle no data perms for model detail page

* Fix loading metadata

* Fix props name

* Make `modelId` optional in `ActionCreator`

* Add `onSubmit` prop to `ActionCreator`

* Allow creating actions with "+ New" button

* Add `nock.cleanAll()`

* Address comments

* Use `CollectionItem` type

* Use `import type`

* Enable loading and error wrapper for loader

* Fix linter

* Make sure we're inlining num in substitution and sum aggregations

* Set hsql version

* Fixing tests

* Fixing tests after the h2 conversion

* Use h2x/extract in h2, fix test differences

* Fixing tests for honeysql2

* Only numbers get inlined

* More test fixes

* Fixing tests

* Undo bad commit

* Inline Ratio must produce doubles or h2 will do integer division where 1/2 = 0

* Fix bad case statement

* We can't run native queries with AS for oracle

---------

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

* use entity loader for db data in actions

* update entity network mock

* add parameter mapping explainer text

* update link

* use styled component link

* update actiondashcardSettings tests

* Make joins work

* Run fuzzy-field-info-test only against H2

* Set driver/*driver* when performing actions

* Restrict expressions+joins-test to DBs supporting date arithmetics

* Fix ordering of (top level) fields in mongo tables

* Explicitly exclude mongo from tests requiring foreign key support

* Optimize away $sort stage for implicit order-by.

* add add question button to empty dashboard

* more options

* allow sidebar toggling

* close navbar on nav to new question

* more emotion

* update tests

* add helpers for actions testing

* add basic action execution test

* update add action test

* add tags and documentation for tags

* add tags to model actions spec

* update test for new actions adding flow

* fix merge conflicts

* fix merge conflict again

* address review comments

* make test more reliable

* prettier

* prettier

Especially helpful for work @camsaul is doing:
https://github.com/clj-kondo/clj-kondo/issues/1996

According to @dpsutton is not used anymore. I wasn’t able to find it on the code as well

* Fix action loading and extraction of action type

* Remove comment

* add dashcard testing mocks

* improve virtual card overlays

* update types

* add search entity types

* add metabase entity support to link cards

* simplify link displays

* update types and support collections and tables

* use edit button to replace entity

* test entity links

* fix rebase

* update types

* address review comments

* handle restricted link entities

* Hydrate link cards info (#28297)

* hydrate link cards info

* we need to hydrate collections too

* Address Cal's comments:
- model use keywrod -> string
- use model hydration
- comments here and there

* Link cards serdes (#28321)

---------

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

* Allow disabling (deleting) basic actions

* Add confirmation modal

* Update how `menuItems` are built

* Use aria-label instead of a test ID

* Tweak confirmation modal copy

* Reinforce test by waiting for results to appear

[ci skip]