* fix missing group by columns from joined tables

* fix spec

* extract model creation helper

* update model actions test to actually mutate data

* fix merge conflicts and clean up

* Keep NFC root fields in nested queries (#29184)

* build strings for 46

* pull in translations for 46

* last-minute updates and add korean

* Query  D E S C R I P T I O N S

* WIP

* MLv2 Query description

* Test fixes

* More robust error handling

* MLv2 more shuffling things around

* Remove unneeded impl

* lib.expressions -> lib.expression

* Address PR feedback

* Fix custom field type aggregation not showing

When the origin table don't have that field type.

* Add tests

* Remove puzzle driven development comments

* Make `getAggregationOperators` API more apparent

* extract common object detail and table settings

* test object detail column settings

It does the wrong thing when passing in a list of :steps.

c.f. discussion at https://metaboat.slack.com/archives/C010L1Z4F9S/p1678905945881399

* db/select-one-field -> t2/select-one-fn

* `db/select-field` -> `t2/select-fn-set`

* db/select-ids -> t2/select-pk-set

* db/select-field->field -> t2/select-fn->fn

* db/select-field->id -> t2/select-fn->pk

* `db/select-id->field` -> `t2/select-pk->field`

* add discourage var

* fix agressive y-axis splitting in charts

* e2e tests

* check if col settings are different

* use `settings.column` instead of redefining default settings

* refactor for feedback

* db/select-one -> t2/select-one

* Move `ActionFormFieldWidget` to action components

* Move `inputTypeHasOptions` to shared utils

* Move `FormFieldEditor` to `FormCreator`

* Clean up `ActionForm` props

* Clean up `FormCreator` props

* Move most of `ActionForm` utils to shared utils

* Move `reorderFields` to `FormCreator`

* Extract `FormCreator` resps from `ActionForm`

* Use default export for `ActionForm`

* Sort `ActionForm's` utils

* Remove not used actions entity constants

* Move `QueryActionContextProvider` to its own dir

* Consolidate `QueryActionContextProvider` utils

* Move `QueryActionEditor` to `QueryActionContext`

* Add `metabase/actions/constants`

* Remove not used export

* Move `isEditableField` to `ActionForm` utils

* Remove not used `hasImplicitActions` utility

* Use `WritebackImplicitQueryAction` type

* Remove `isImplicitAction` utility

* Move `shouldPrefetchValues` to parameters form

* Fix typo

* Move `getSubmitButtonColor` to shared utils

* Move `getSubmitButtonLabel` to shared utils

* Sort `ActionParametersInputForm` imports

* Move `reorderFields` to `ActionForm`

* Sort `metabase/actions/utils` imports

* Sort imports

* `db/select-one-id` -> `t2/select-one-pk`

* add discourage var

* db/select-one-field -> t2/select-one-fn

* add discourage var

Exposes models as a GenericTable and adds in UI menu option to show the x-ray. There's still work to be done to add filters and more interesting auto-generated charts, but this is the first step.

Minor point version updates, including the redshift driver.

* Transform card data into the same format as that which comes out of the DB

* Use `from-json` instead of toucan transformations

* Remove unused require

* Update docstring and rename to from-parsed-json

* Oops, fixed

* Remove now

* Add docstring

* Fix order of docstring

* Don't update created_at and updated_at

* Extract `parseArguments` into utils

* Allow custom browser for Cypress snapshots generation

* [CI] Do not run `i18n` workflow on spec changes

* Reverse the order of `name` and `run`

* Updating graph.metrics when column is removed

* updating logic and adding unit tests

* Ensure we use the ssh tunnel on action execution

I'm gonna see if we have an existing way to run tests against ssh dbs. I
see there are some tests that use a faking ssh server. Not sure if those
are useful or not. But otherwise ssh stuff needs out of process help to
accomplish so not sure if it's easily feasible right now

* introduce new `with-unpooled-connection-spec` macro

it is identical to the `with-connection-spec-for-testing-connection`
macro but has a better name. In the future they could diverge as testing
requirements and action requirements change. But we want that
_implementation_ (which correctly uses tunnels), but we do not want a
"testing connection" slug in the usage.

In the future (or now if we prefer a different name) we can rename it,
find the usages that are not testing vs the ones that are, etc.

* tests for actions over ssh

* unused require of mb.u.ssh

had brought it in but then stuffed all of this inside of
`with-unpooled-connection-spec` so no need to worry about ssh, closing
it, ensuring the port is there, etc.

* Don't test h2 with ssh tunnel

details lack a host and it npe's in the ssh tunnel stuff. No big loss as
ssh on h2 is not supported (due to this error, and probably many more)

* Switch to toucan2 for db access in tests

* Use connection pool for custom actions connection

Was originally using `jdbc/get-connection` from the details of the
db. This was a one-off non-pooled connection that also did not reuse ssh
tunnels. I thought the one-off connection was important since it is not
read only, but that's not the case. So we can reuse the exact same
mechanism that the implicit actions are using with
`metabase.driver.sql-jdbc.actions/with-jdbc-transaction`.

* cyclic dependency

almost moved it into execute, but that gives a very inviting
`sql-jdbc.execute/with-jdbc-transaction`. Welcoming even. But
`sql-jdbc.actions/with-jdbc-transaction` lets you know its only for
transactions.

* Add filter constructor functions

* Revert changes to defop

* Address linter complaints

* Add function to set the query filter

* Address linter complaints

* Fix test name

* Add an aggregate function to add aggregations to queries

* Call ->op-arg on aggregate input

* Allow any in aggregations for now because expressions on aggregates are allowed

* We have to recurse into clauses

* add permissions and test

* address comments

* fix whitespace

* remove ehitespace

* revert vscode changes