* Use type/JSON instead of type/SerializedJSON

* Tidy migration

* Update migration

* Fix rollback migration for h2

* whitespace

* Add test for migration

* Fix test

* Add rollback

* whitespace

* Test JSONB type as well as JSON

* Don't fingerprint JSON columns

* Remove comment, that might be wrong in the future

* Update test

* Add tests for fingerprinting

* Use base-type JSON for fingerprinting base query

* Add test for visibility-type=details-only

* undo .

* Change migration id

* Fix migration test

* Merge master

* Exclude mariadb from tests

* Make is-mariadb? public

* convert db\/insert! to t2

* Drop support for nested field columns for redshift

* db/select -> t2/select

* Remove clj_kondo file

* Add action.dataset_query to search results

* Add action.database_id to search results

* Unskip repro

* better FE error handling for action query display

* update repro

---------

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

* Remove usages of `cy.addFilterToDashboard` custom command

* Fully remove `cy.addFilterToDashboard` custom command

They were getting deleted, but only when granting permissions.

Now they will be deleted whenever their v1 path counterpart gets deleted.

* WIP improvement of recent_views endpoint query

* Adjust query to actually include models (cards with :dataset true)

This was not part of the recent_views before, so I think it's actually an improvement to the feature as well as
improvement to the characteristics of the query itself.

* Linter fixes

* One more take on recent_views query. Last 30 days, all req's in 1 query

This query combines view_log and query_execution
Keep query_executions when they are card with context 'question'
Keep any card views when they are 'datasets' (aka models)
Remove duplicates with GROUP BY model, model_id
Sort by timestamp desceneding
Remove any views that are bookmarked by using NOT IN

* An improved query that handles all logic in one db query

* Adjust limit in case archived cards/dashboards are in the list

* Make fn private for linter

* Add one additional constraint to prevent pinned items from counting as viewed

* Address review feedback

* Accidentally slipped this in... not needed on this PR

* fix color range for null values in table

* add another test case

* initial changes

* styling change

* address comments

* fix sso settings

* update test

* update test

* remove warning

* fix test

It's broken there because it tries to capture the Var returned by
`defn` in a `let`. Vars aren't real in CLJS, so this doesn't work.
The compile succeeds but the JS output is subtly broken.

* show all subscriptions in notification management page regardless of collection perms

* strip cards and recipients if necessary

* indentation tweak

* add backend tests

* add cypress repro

* fix permissions check

* slightly different approach

* fix cypress test

* fix test

* address comment

* MLv1 remove unused

* Remove definitely unused methods

* Use internal metadata for `generateQueryDescription`

* Remove `tableMetadata` from `SaveQuestionModal`

* Remove `tableMetadata` from `AggregationPopover`

* Remove `tableMetadata` from `QueryBuilder`

* Remove console.logs

* Fix `_getMLv2Query` call

* Don't fingerprint fields that have (isa? :base_type :type/Structured)

* Add fingerprinting test

* Add function for updating filters

* Support adding and getting filters

* Add current-filters function for handling a conjunction of filters

* fix dashboard opening in editing state when it shouldn't

* unskip reproduction test

* Make `ActionForm` accept an `action` prop

* Make field settings inside `ActionForm` if missing

* Clean date-time values inside `ActionForm`

* WIP

* Clean and filter values on submit in `ActionForm`

* Sort `ActionVizForm` props

* Sort `Action` props

* Use more generic `WritebackAction` type

* Move more logic into the `ActionForm`

* Add `useActionForm` hook

* Fix parameter type in tests

* Fix submitting a form when all parameters mapped

* Move utils to `useActionForm`

* Fix submit button label in public action e2e tests

* Filter changed values only for implicit update actions

* Rename `cleanInitialValues` to `cleanedInitialValues`

* Fix `shouldShowConfirmation` can be a string

* Nice boring code

* Fix types in tests

* Small rename

* Move `useActionForm` utils closer to source (#29328)

* Fix `ActionForm` ignored submit errors

* Fix default values display in the form editor

* Fix names

db/update! -> t2/update!

* 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

* Cumulative FE changes

* WIP

* lib.expressions -> lib.expression

* Address PR feedback

* Fix everything

* Oops fix missing metadata

* Test fixes 

* Fixes post-merge

* Fix merge conflict

* JS metadata overhaul

* Test fix 

* Test fix

* Hack to workaround busted JOINs

* Use Braden's better fix

* Cumulative fixes

* Test fixes 

* Remove unused

* Include temporal unit in query description

* Don't change indentation

* Revert e2e browser changes

* Revert bin change

* Meaningless change to kick CI