* Squash

* Replace connection-with-timezone usages

* Add deprecation warning.

* Fix docstring

* Collection to toucan2

* Permissions to toucan2

* PermissionsGroup to toucan2

* PermissionsGroupMembership to toucan2

* PermissionsRevision to toucan2

* GroupTableAccessPolicy to toucan2

* ApplicationPermissionsRevision to toucan2

* CollectionPermissionGraphRevision to toucan2 too

* load snapshot instance data

* stop hardcoding question/1

* parameterize default question IDs

* cleanup

* fix imports

* more slashes

* misc fixes

---------

Co-authored-by: Bryan Maass <bryan.maass@gmail.com>

* Implement `suggested-join-condition`

* Update docstring

* Calculate and set join alias when joining

* Use BuildJet runner for `be-cloverage`

We're seeing OOM failures in `be-linter-cloverage` job in CI.

* revert back to using ubuntu-22.04 and increase heap size to 2GB

* 2gb to 4gb

* add comment

---------

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

* update data reference

* update screenshot

Resolves #28792 

* Always create a transient dataset for BigQuery tests

* Simplifies BigQuery test data code

* More BigQuery test data code cleanup

* More BigQuery test data code cleanup, test fix

* Extracts test-db-name in bigquery tests, restores normalize-name

* Fixes BigQuery QP tests for new test DB name scheme

* Removes extraneous clojure.core

* Updates bigquery QP test helper code style

* Cleans up comments and code in bigquery test data

* Update modules/drivers/bigquery-cloud-sdk/test/metabase/driver/bigquery_cloud_sdk/query_processor_test.clj

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

---------

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

[ci skip]

I forgot to do this in #31432.
As the result, some PRs have been stuck in CI due to the required check.

This PR will run the fallback job for `actions` E2E group when changed
files don't require E2E tests to run.

[ci skip]

* WIP showing Toast

* add dismissed_toasts stub on GET api/user/current

* backend to save dismissed toast names as user-local-settings

- adds dismissed_toasts key to GET user/current
- introduces POST /user/:user-id/dismissed/:toast-name
  - For now, only custom_homepage_changed is a valid `toast-name`, but
    it can be extended to others.
  - I nearly put in a DELETE to unset it, which could be useful for
    tests, but took it out.

* add untested DELETE route to ease devtime pains

* setup dismissed-custom-dashboard-toast to be "yet another setting"

- remove specific endpoints
- keep the result on GET api/user/current

* dismissed_toasts + improve setting api to take Snake Case

- with tests

* Adding dismiss action

* dismissed toast map

* Ensure that toast is dismissed for admin who set the custom homepage property

* Fixing 2 toasts on login

* Moving dismissed toast to settings

* updateSettings, undo.canDismiss

* adding loading state to settings, removing unneeded CLJ tests


---------

Co-authored-by: Bryan Maass <bryan.maass@gmail.com>

* User to toucan2

* Pulse to toucan2

* PulseCard to toucan 2

* PulseChannel and PulseChannelRecipient to toucan2

* Query, QueryCache, QueryExecution to toucan2

* Metric, Segment, MetricImportantField to toucan2

* ParameterCard to toucan2

* DataMigrations and FakeCard to toucan2

* NativeQuerySnippet to toucan2

* Disable slow test to prevent flakiness

Fixes #29266.

This should not really happen, but parts of the code handle this
situation while others don't.

Due to AOT (Ahead Of Time) compilation (specifically direct linking),
the `MB_COLORIZE_LOGS` and `MB_EMOJI_IN_LOGS` env vars where not
respected.

Marking `metabase.util.format/colorize` and `metabase.util/emoji` with
`^:redef` metadata prevents the compiler from using direct linking for
calls to these functions (and thus evaluating the env vars at compile
time) and thus delays resolution of what function to call until runtime
when the actual user-provided value of the env var is known.
cf. https://clojure.org/reference/compilation#directlinking

Closes: https://github.com/metabase/metabase/issues/30518

Also make sure that the attached columns have the right temporal bucket information.

* Filter import on YAML extension

* sorts namespaces

---------

Co-authored-by: Luiz Arakaki <luiz.arakaki@metabase.com>

* Avoid expensive call to fetch DB metadata

Double-checking the schema name is unnecessarily defensive since we
already call include-schema?

* Check that schema is set

Previously: one “feature or project” template with the `.Epic` label automatically attached.
Now: two templates, one for epics, one for smaller things that don't require milestones (or `.Epic` label; they get `.Task` instead).

Squashed commit messages follow:

* Remove unused optional field

* Update title to current convention

* Fix typo

* Remove unused section

* Change capitalization for consistency

* Remove redundant description

* Rename the issue template to mention epic

* Shorten the description of the issue template

* Lighten up the milestones section

Unfortunately, it doesn't seem possible to prepopulate the issue with an
empty tasklist (https://github.com/orgs/community/discussions/50502).

* Add an issue template for tasks [ci skip]

We had one for epics, with milestones and `.Epic` label. Pretty often
that label was accidentally left on smaller issues that were created
using the same template. Now the smaller issues have their own template
and their own label. Yay.

https://metaboat.slack.com/archives/C04CGFHCJDD/p1686548875239219

* Remove redundant angle brackets

* Use a string to get a valid yaml file

Without the quotes, github says
`Error in user YAML: (<unknown>): did not find expected key while parsing
a block mapping at line 1 column 1`

* Remove rarely if ever used placeholders

* Call out optional-ness of some links

* Contextualize a header

Context can be some text, a slack thread, a notion page, another issue,
whatever.

* update audit log view migration

* separate mysql and fix h2

* typo

* fix views

---------

Co-authored-by: Luiz Arakaki <luiz.arakaki@metabase.com>

* Fix dashboard description markdown overflow

* Review fixes, Improved e2e test

* Fetch field icon from dimension instead of semantic_type (#28622)

* simplify field.dimension().icon() → field.icon()

* resync tooltip to reflect non-semantic icon

* add ts-ignore to iconName with justification

* Updating eslint rules to allow importing types to metabase-lib

* Revert "Updating eslint rules to allow importing types to metabase-lib"

This reverts commit 1b6aca4362a7ca530046466bc9c1bfc02d5b116c.

---------

Co-authored-by: Abdul Aziz Ali <abdulazizali@acm.org>
Co-authored-by: Nick Fitzpatrick <nickfitz.582@gmail.com>

* Add `BreakoutClause` type

* Extend `displayInfo` for breakouts

* Add wrappers for `breakouts` and `breakout`

* Fix property names in breakout clause types

* Add TypeScript wrappers for temporal buckets

* Add basic `ColumnPrecisionPickerPopover`

* Add TypeScript wrappers for binning strategies

* Add `BinningStrategyPickerPopover`

* Rename to `TemporalBucketPickerPopover`

* Remove type properties from breakout clause type

* Unify temporal bucket and binning strategy types

* Unify bucket pickers

* Fix `isCalculated` property name

* Fix bucket methods usage in Jest

* Allow calling `displayInfo` with `Clause`

* Add `isClauseColumn` helper

* Make bucket pickers optional in `QueryColumnPicker`

* Highlight selected column in `QueryColumnPicker`

* Add a11y labels to bucket pickers

* Use `createMockMetadata` in notebook tests

* Use MLv2 in `BreakoutStep`

* Update `SortStep`

* Fix stage index for `BreakoutStep`

* Tweak bucket types

* Use named exports for bucket popovers

* Format temporal bucket name in the column picker

* Update `binning` and `temporalBucket` types

* Add `isSameBucket` utility

* Tweak the column picker

* highlight selected bucket
* prevent unexpected bucket reset

* Use temporal buckets JS wrappers

* Use default bucket in the column picker

* Fix temporal bucket test

* Add test IDs for compatibility with e2e tests

* Fix "Unbinned" bucket handling

* Fix e2e test to use a11y selectors

* Add long-display-name to display-info

* Use long display name in the notebook

* Skip getting metadata from nil

* Handle JS objects as fingerprint metadata

* Set max height for bucket picker popover

* Fix `summarization.cy.spec` test

* Fix `joins.cy.spec` test

* Attempt to fix MLv2 query validation

* Test that integer expressions are recognized by isa/numeric?

* Make stage index explicit

* Fix custom-columns e2e tests

* Fix breakout validation

* Remove `React` imports

* Break down `hasBucketing` prop

* Keep the metadata from the tables list for questions (#31162)

* Keep the metadata from the tables list for questions

Previously, the metadata loaded from "questions" would overwrite
the metadata loaded from "tables" if both existed. This PR changes
that to merge the information from the two sources instead.

* Skip using _plainObject when parsing cards from table metadata

* Format saved question display names in joins as "Question <card-id>"

* Deal with field-ref shaped field IDs in the JS provided metadata

* Fix long display name source question breakout (#31312)

* Fix icon import

* Fix e2e test

* Fix e2e test

* Allow field references by name

* Make sure no temporal buckets and binning strategies are offered for expressions

* Fix MLv2 tests

* Return empty vector for visible-columns on nil

* Add `base-type` to MLv1 compatible field rfs

* Use `selected` flag on buckets

* Recognize expression columns when marking selected breakouts

* Add backend regression test for expression breakout column handling

* Test fix

* Rework column bucket pickers

* delegate more responsibilities to binning/temporal pickers
* include "Don't bin" option on FE vs BE

* Fix e2e test

* Rework bucket pickers 2

Forgot to push this in the first place 



* Fix redundant eslint-disable comment

* Use *-position to check if a column is selected

* Remove dont-bin pseudo binning strategy (#31443)

* Fix matching breakout columns with binning (#31441)

* Stop filtering out "Don't bin" on the FE

* Fix #31482 not showing join display name for busted breakout from drill thru

* MLv2: Fix matching of field literals to field integer IDs (#31487)

* MLv2: Fix matching of field literals to field integer IDs

* Better test

* Update src/metabase/lib/equality.cljc

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

---------

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

* Remove infinite recursion when resolving fields by name (#31485)

* Remove infinite recursion when resolving fields by name

* Remove stray tap>

---------

Co-authored-by: Case Nelson <case@metabase.com>
Co-authored-by: Tamás Benkő <tamas@metabase.com>
Co-authored-by: metamben <103100869+metamben@users.noreply.github.com>
Co-authored-by: Cam Saul <github@camsaul.com>
Co-authored-by: Cam Saul <1455846+camsaul@users.noreply.github.com>

* fix funnel tooltip artifacts in safari

* force composite-layer instead of forceRedraw

* used styled components

---------

Co-authored-by: Aleksandr Lesnenko <alxnddr@gmail.com>

We're trying to minimize the amount of hard coded IDs (or any other data) in E2E tests. This change doesn't assume admin's ID is always going to be 1. It fetches the id from the current user instead.

This should make test more resilient to change.

* add the migrations

* edit comment

* add 'if exists' to drop view statements

* update yaml: move sql to the right spot

* Extract `questionDetails`

* Move `public-sharing` to the `sharing` folder

* Remove redundant test

This part was covered in `public-sharing.cy.spec.js`.

* Create a separate `public-dashboard` E2E spec

- Extract relevant part from `public.cy.spec.js`
- Make all tests in `public-dashboard.cy.spec.js` run in isolation

* Make tests in `public.cy.spec.js` run in isolation

* Remove redundant wait

* Limit the query results to speed test up

* Merge public questions E2E tests

* Merge `downloads` with the `sharing` E2E group

* Remove `downloads` from the folder list

* Refactor `public-dashboard.cy.spec.js`

- Clean up linter warnings by using semantic selectors
- Speed test up by using API

* Add snowplow event tracking

* Use integer type for model_id and size_mb instead of size_bytes

* Rename upload_speed to upload_seconds and avoid wasting time if the upload was successful