* Revert "Fix zoom in drilldown for FKs (#28022)" (#29551)

---------

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>
Co-authored-by: Nemanja <31325167+nemanjaglumac@users.noreply.github.com>

* keep standalone values when collapsing column

* adjusting unit test

* PR Feedback

* adding space

* removing only

* Adjusting behavior and tests

* adjusting data_grid unit tests

Co-authored-by: Nick Fitzpatrick <nick@metabase.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Mark Bastian <markbastian@gmail.com>

Co-authored-by: Denis Berezin <denis.berezin@metabase.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Natalie <nat@metabase.com>

* Separate date from datetime binning options

The BE sends over binning indexes for each field and a top level map for
how to interpret these keys.

```javascript
❯ http get http://localhost:3000/api/table/10658/query_metadata

 Cookie:$SESSION -pb
{
    "active": true,
    "caveats": null,
    "created_at": "2023-03-21T22:20:42.363169Z",
    "db": {
        "engine": "postgres",
        "features": [ "full-join", ... ],
        "id": 499,
        ...
    },
    "db_id": 499,
    "dimension_options": {
        "0": { "mbql": [ "field", null, { "temporal-unit": "day" } ],
               "name": "Day",
               "type": "type/Date"},
        "1": { "mbql": [ "field", null, { "temporal-unit": "week" } ],
               "name": "Week",
               "type": "type/Date" },
        "10": { "mbql": ["field", null, { "temporal-unit": "quarter-of-year"}],
                "name": "Quarter of Year",
                "type": "type/Date" },
        "11": { "mbql": [ "field", null, { "temporal-unit": "minute" } ],
                "name": "Minute",
                "type": "type/DateTime"},
        "12": { "mbql": [ "field", null, { "temporal-unit": "hour"}],
                "name": "Hour",
                "type": "type/DateTime"},
        ... // more options. They have a type, an id, and an mbql clause
    },
    "display_name": "Different Times",
    "entity_type": null,
    "fields": [{"name": "id",},
               {"name": "dt",
                "dimension_options": [
                    "11", "12", "13", "14", "15", "16", "17",
                    "18", "19", "20", "21", "22", "23", "24", "25"]
                "database_type": "timestamp",
                "effective_type": "type/DateTime",
               },
               {"name": "t",
                "dimension_options": [ "26", "27", "28" ],
                "effective_type": "type/Time",
                "database_type": "time"
               },
               {"name": "d",
                "database_type": "date",
                "dimension_options": [ "0", "1", "2", "3", "4", "5",
                                       "6", "7", "8", "9", "10" ],
                "effective_type": "type/Date",
        }
    ],
    "name": "different_times",
    ...
}
```

This PR adds some new types just for dates so that you don't get offered
by group a date by hour or minute.

This adds new indexes to this list, which makes us wary that these
indexes have leaked out and cannot be changed. But that is not the
case. This is essentially compression on the response. When the FE saves
a query, it does not include the index but uses the mbql query to insert
the proper temporal-unit:

```clojure
;; a "date" field choosing the quarter option
{:type :query,
 :query {:source-table 5,
         :breakout [[:field 49 {:temporal-unit :quarter}]],
         :aggregation [[:count]]},
 :database 1}

;; the same query but binning by week
{:database 1,
 :query {:source-table 5,
         :breakout [[:field 49 {:temporal-unit :week}]],
         :aggregation [[:count]]},
 :type :query}
```

So the index is relative to just the editing session of the query, and
the indexes do not have to be stable when we add new ones.

* athena and mongo (others?) don't have a date type

the test expects a date field to have date binning options. previous, we
made no distinction for binning options for date vs datetime fields. Now
we do and have to manage expectations in the tests.

* mongo returns type/Instant

Co-authored-by: dpsutton <dan@dpsutton.com>

* add permissions and test

* address comments

* fix whitespace

* remove ehitespace

* revert vscode changes

Co-authored-by: Jerry Huang <34140255+qwef@users.noreply.github.com>

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

Co-authored-by: Natalie <nat@metabase.com>

Co-authored-by: Nick Fitzpatrick <nick@metabase.com>

* Improvement of recent_views endpoint query (#29259)

* 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

* Add toucan2 require

---------

Co-authored-by: adam-james <21064735+adam-james-v@users.noreply.github.com>
Co-authored-by: Adam James <adam.vermeer2@gmail.com>

* Break down `ActionForm` component (#29290)

* 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`

* Consolidate action form logic (#29326)

* 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

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

* 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: Tim Macdonald <tim@metabase.com>
Co-authored-by: Ryan Laurie <iethree@gmail.com>

* Remove usages of `cy.addFilterToDashboard` custom command

* Fully remove `cy.addFilterToDashboard` custom command

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

Co-authored-by: Natalie <nat@metabase.com>

* Actions/e2e model actions tests (#29295)

* extract model creation helper

* update model actions test to actually mutate data

* fix merge conflicts and clean up
# Conflicts:
#	e2e/test/scenarios/models/model-actions.cy.spec.js

* don't assert not found

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

* 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

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

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

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

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

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

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

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

* make object detail work for non-logged-in users

* update types to account for optional params

* test public object detail viz

Co-authored-by: Ryan Laurie <30528226+iethree@users.noreply.github.com>

* fix missing group by columns from joined tables

* fix spec

Co-authored-by: Aleksandr Lesnenko <alxnddr@users.noreply.github.com>

* extract common object detail and table settings

* test object detail column settings

Co-authored-by: Ryan Laurie <30528226+iethree@users.noreply.github.com>

* build strings for 46

* pull in translations for 46

* last-minute updates and add korean

Co-authored-by: Maz Ameli <maz@metabase.com>

* 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

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

Co-authored-by: Emmad Usmani <emmadusmani@berkeley.edu>

* 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

Co-authored-by: Aleksandr Lesnenko <alxnddr@users.noreply.github.com>

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