[ci skip]

* Mark `moment-timezone` as deprecated

* Fix other eslint configs

* Disable eslint for current occurrence of moment

* Ensure that field picker icon does not get smushed

* removing unhelpful test

* register visualizations in test

* use replace for programatic navigation

* adding comments

* red: reproduce the error in test case

* green: return id in removeParameter action

* refactor: create dashboard actions parameters unit test

* chore: rename unit test

* fix: update test's issue reference

* chore: add new line

Fix - Unsaved changes warning shown when going back to notebook after visualizing the question (#35003)

* Fix running and then editing the question

* Fix running and then editing the structured model

* Add a test for #35000

* Add a test for models in #35000

* Consolidate waiting for save to be enabled/disabled
- Rename waitForSaveQuestionToBeEnabled to waitForSaveToBeEnabled
- Rename waitForSaveModelToBeEnabled to waitForSaveChangesToBeEnabled
- Rename waitForSaveModelToBeDisabled to waitForSaveChangesToBeDisabled
- Remove waitForSaveNewQuestionToBeEnabled in favor of waitForSaveToBeEnabled

Previously only the `:column-name` was provided, and there is a
self-proclaimed "icky hack" to guess at the full columns. The FE
provides the (JSON) columns, so this PR converts and uses them.

The icky hack can be removed.

`:column-ref` is included because sometimes it's important that the refs
use the same UUID.

Fixes #34729.

EOL v5.7 was October 21, 2203.

The image changed from the legacy `circleci/` image to `cimg/`
https://circleci.com/developer/images/image/cimg/mysql

- Some tests had a redundant initial route just so that there is
  an entry in history we can go back to. This caused NewItemMenu to
  be redundantly rendered, which caused race conditions in the entity
  framework. We do not need history.goBack(), we can use history.push()
  but we need an extra, safe route we can try to navigate to.
- @see https://metaboat.slack.com/archives/C057T1QTB3L/p1697183111294489
- @see https://metaboat.slack.com/archives/C5XHN8GLW/p1697811520881169
- @see https://github.com/metabase/metabase/pull/34661

* Move E2E download uberjar logic into a composite action

* Check out only the last 20 commits

* Always use the correct, last commit in a branch

This helps us avoid the slippery slope in which GitHub creates a new
merge commit, that doesn't exist in the repo, under our feet.

* Adapt the stress-test workflow to use new download action

* Increase the number of commits to search through

* Update .github/actions/e2e-download-uberjar/action.yml

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

* Log full artifact name

* Correctly download artifact from the current build

---------

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

[ci skip]

This reverts commit 45d74606.

[ci skip]

* Extract `FilterHeaderButton`

* Move `QuestionFilters` to its own directory

* Remove not used `QuestionFilters` component

* Extract styled components

* Convert `ViewPill` to TypeScript

* Convert `QuestionFilters` to TypeScript

* Clean up

* Fix label

No need to repeat the same line for every single **step**
when all steps can read from the **job** context.

Add new sqlserver 2022 jobs to the CI and update the existing one's name.

Makes /api/properties and /api/premium-features/token/status return valid structures with information about invalid token, plus logs the error about token being invalid in logs. Also this information is used to show a PaymentBanner on top of the screen so that admins can understand what is going on.

Resolves #33858

* PoC for explicitization of dynamic vars

* deref the current-user delay

* missed one

* fix flaky log test

* create-card! takes a card as first arg