* 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

* Add dashboard parameter mapping for date range

* Fix unit tests

* Add e2e tests

* Update logic to use date range only for specific units

* Add Mantine `Popover`

* Fix spacing

* Deprecate previous popovers

* Update isNavigationAllowed logic to prevent new model creation

* Add simple model creation test

* Remove identifier

* Update comment

* Add a test case for saving a new model

* Avoid using fetchMock directly in QueryBuilder unit tests

* Remove setupCollectionsEndpointsByIds in favor of setupCollectionByIdEndpoint

* Add native question test base

* Update isNavigationAllowed tests

* Make isNavigationAllowed tests pass

* Update shouldShowUnsavedChangesWarningForSqlQuery logic to account for new questions

* Add a test case to run new native question

* Update mock to include individual collection GET endpoint

* Add DataSourceSelectors to MockNativeQueryEditor

* Update test beforeunload event to reflect new behavior

* Use setupCollectionByIdEndpoint instead of modifying setupCollectionsEndpoints

* Update failing test

* Use serializeCardForUrl

* Add a test for native query editing

* Do not trigger warning when leaving empty question

* Add beforeunload tests for creating new questions

* Remove redundant div

* Make assertions consistent

* Use single resetAllMocks()

* Fix post-rebase conflict

* Simplify getShouldShowUnsavedChangesWarning's logic

* Add isModifiedFromNotebook to store

* Show unsaved changes warning when structured question has been modified via notebook editor

* Use explicit return type, as it's not obvious that this function needs to be a Promise

* Remove unused selector dependency

* Update mocks with new attribute

* Add a test for editing notebook questions

* Extract waitForSaveNewQuestionToBeEnabled out of triggerNotebookQueryChange

* Introduce waitForSaveModelToBeEnabled & waitForSaveModelToBeDisabled

* Add a test case

* Remove redundant navigation

* Update isNavigationAllowed logic and tests

* Refactor tests and add more test cases

* Add more test cases

* Shorten code

* Add a test for saving the question

* Refactor isNavigationAllowed

* Add a test for not showing modal

* Add a test for non-notebook changes

* Organize code

* Use existing setUIControls

* Add extra assertions for post-save navigation to check if question is properly mark as non-dirty

* Add extra assertions for post-save navigation to check if question is properly mark as non-dirty

* Add a post-save nagivation test

* Add logic to reset UI controls isModifiedFromNotebook state after saving the question

* Rename to allowedPathnames

* Add explanatory comment

* Update failing e2e test

* Fix #32963 e2e repro failing

* Allow `/model/${slug}` to make models-query-editor e2e test pass

* Update isNavigationAllowed logic to make e2e tests pass

* Update isNavigationAllowed tests

* Add 1 more model location

* Revert identifier name change

* Use optional chaining operator for brevity

* Fix typo

* Rename notebook to structured

* Rename getQuestionLocations to getNativeQuestionLocations and don't use getNativeQuestionLocations in getStructuredQuestionLocations

Moves `migrate-click-through` and `migrate-remove-admin-from-group-mapping-if-needed` to custom migrations, so that all migrations are handled by liquibase.

Resolves #32611, #32612

Co-authored-by: John Swanson <john.d.swanson@gmail.com>

* remove redundant cy.wait

* try again to fix test failure in CI that doesn't repro locally

We want to add some tests to verify that this (slightly hidden) feature continues working.

Fixes: https://github.com/metabase/metabase/issues/21163