* Revert changes from Jeff's PR

* Add index to ModerationReview moderated_item_type + moderated_item_id

* Add to 0.41.2 instead

* Require explicit index name for createIndex

* Move 41.2 migrations to after the 41.0 migrations

* Adopt new migration numbering scheme

* Fix comments

* Fix MySQL + MariaDB insanity

* Fix ID range validation

* Actually 382 is the last legacy ID

* Improved validation and tests

* Adopt the new-new migration ID format.

* Test fixes 

* Adopt new migration numbering scheme

* Fix comments

* Fix MySQL + MariaDB insanity

* Fix ID range validation

* Actually 382 is the last legacy ID

* Improved validation and tests

* Adopt the new-new migration ID format.

* Test fixes 

* Add unit tests for schema utils

* Add data bucket step to DataSelector

* Go to next steps from the data bucket step

* Add different type of AccordionList icons layout

* Add back navigation from 'Raw Data' tab

* Fix selecting a database in 'Raw Data'

* Enable search on data bucket step

* Only render data bucket step if there are datasets

* Prevent opening the data bucket step without datasets

* Update picker search placeholder text

* Update searchable models

* Fix displaying datasets in search results

* Hide "Saved Questions" in "Raw Data" tab

* Encode data in schema str

* Add ability to request dataset virtual tables

* Support datasets in SavedQuestionPicker

* Use saved question picker for datasets too

* Add E2E tests

* Show dataset icon in search results

* Fix E2E selectors

* Fix AccordionList icon rendering

* Fix dataset can't be selected in search

* Fix data selector step hydration

* Fix JoinStep unit tests

* Fix dataset icon size in picker view

* Fix schema picker without datasets

* Fix DataSelector tests

* Fix error about null data selector step

* Move schema utils to `metabase/lib`

* Fix questions don't appear in pickers after created

* Fix JoinStep unit tests

* Test can create a question based on a dataset

* Simplify schema ID parsing

* Fix typo

* Fix search models

* Fix ellipsis character

* Fix "Raw Data" button click area

* Remove temp comment

* Fix URL in personal collection contents request

* Add visual tests for table visualisation

* Wait until spinner disappears

https://www.metabase.com/blog/Metabase-0.41/index.html

* fix expression dimension failing to find underlying field

* add a repro

* move querystring-syncing logic to new hook

* rename <Parameters> to <SyncedParametersList>

* Remove syncQueryString prop

* remove unused 'query' prop

* remove 'isQB' prop

* remove ParametersWidget in favor of ParametersList handling empty parameters list

* Rely on value-populated parameters objects over parameterValues map

* remove unused pass-through prop 'vertical'

* make props passed from Dashboard to SyncedParametersList explicit

* Update name of component

* create a more generic hook for syncing the url query string

With the suggestions and highlighting being done through via lexical analysis
only, there is no need to run the computationally-expensive syntactical
analysis on every keystroke anymore. We only need to do that when the
user finishes the editing and wants to use the expression.

* Fix typo

* Don't warn about unused variables prefixed with _

* Fix unused var warning for Icon component

* Add `varsIgnorePattern` for JS files too

* Move the multimethods over to strings not classes

datasets are Cards so don't have a good way to intercept. And this kinda
took all of the keywords, turned them into classes, and then didn't
actually use those classes-- they were just opaque keys in the
hierarchy. No bueno.

* Include datasets in search results

* Docstring for the linter. Thank you clj-kondo

* Fix namespace decls

* Update tests

* Clean up ns and make test more resilient

these tests that test global collections can be sensitive to test order
and the CI dbs that continually exist rather than be
recreated. Sometimes "table" shows up in these available models. Kinda
don't care so just check for subset

* Rename for confusion

* tree view for datasets

* Update docstring

There were global audit query limits which defaulted to 1000 previously. This makes them less limited.

* Expose the e2e downloads helper function

* Apply the new e2e downloads helper to the #18382 repro

* Apply the new e2e downloads helper to the #18440 repro

* Update the helper so it applies both for saved and unsaved questions

* Add additional helper options

* Redirect to a non-existing page

* Reafactor `downloads` spec

* Extract repro for #10803

* Move UI mocks into separate file

* Mock ace and Google Analytics by default

* Add helper to run a test in EE environment

* Add test ID to CacheTTLField

* Migrate tests to new EE helper

Delete Secret instances from the app DB, when the corresponding Database instance is deleted

Updating existing test (and renamed it to `secret-db-details-integration-test`) to confirm this behavior

* fix invalid js

* install typings using typesync

* upgrade classnames to get its own typings

* convert colors, settings, Icon to TS

The possible matching/choices for function names, dimensions, segments,
and metrics are only shown when there is at least ONE matching character.
Thus, when the user hasn't inputted anything yet, there won't be any
suggestions.

* Repro #18630

* Ignore field literals in Join's isValid check

* Don't use Join's `isValid` method at `clean`

* Update the keys we consider to unverify a card

Previously when almost anything changed we would unverify a card. But
this was way too aggresive. We really only want to unverify when a query
changes. Far simpler to specify the keys we do want to consider rather
than the keys to ignore.

Was prompted because when we marked a question as a dataset it would
unverify because the `:display` was changed and also `:dataset`. Neither
of these matter for verification purposes

* Update testing context string [ci skip]

* Empty commit to trigger CI

had skipped ci since it was a docstring change but that prevents merging
sad face :(

`org.apache.sshd` requires `net.i2p.crypto/eddsa` dependency
to work with ED25519 keys.

This worked in 0.40 because `eddsa` was included as a transitive
dependency of another unrelated dependency (`buddy`) but stopped
working in 0.41 because the new version of `buddy` no longer depends
on `eddsa`. Thus, we must explicitly include the dependency.

To prevent this from breaking again, switched one of the test
keys to an ED25519 key.