* Add repro for #25031

* Rename file

* Clean up issue references

This reverts commit 971c5053.

* pivot table conditional formatting

* tweak styles

* hide highlight row setting

* adjust styles

* specs

* Extract `MainNavbarProps` type

* Extract `MainNavbar` component

Would be responsible for picking between:

* regular navigation sidebar content
* launched app navigation sidebar content

* Extract `NavbarLoadingView` component

* Use different URLs for launched and previewed app

* Add basic "Launch app" button

* Allow `SidebarLinks` without `icon` prop`

* Add basic app navigation sidebar

* Fix opening dashboards inside apps

* Lift `selectedItems` getter up to `MainNavBar`

* Highlight open data app page

* Unify `SidebarLink` components

* Add "Exit app" button

* Move `cmd + .` shortcut handler to `MainNavbar`

* Fix opening regular dashboards

* Add "rocket" icon

* Use "rocket" icon for "Launch app" button

* Update data app URLs

* Consolidate data app URL helpers

* Fix typescript error

* Remove not used regexp

* Move `CollectionCreate` back to `MainNavbarContainer`

* Add settings action buttons

* Add missing `ul` tag

* Simplify `LaunchDataAppButton`

* Hide "Launch app" button when app is launched

* Open app once it's created

* Turn "Exit app" button into a Link

* Fix type error

* Use `import type` syntax

* Add disable-max-results? to query middleware options when refreshing cached card results

* Add docstrings

* Add test for `add-default-limit`

* [App] Add action_id to dashcard and endpoint for execution

With app forms, we will be inlining action execution. So this PR is in
preparation for that inlining and removing the need for createing
Emitters in order to execute actions.

Mirrors card querying from dashcards. This expects parameter_mappings
of the dashboardcard to hold the mappings for execution, in the same
shape as other dashcards. Instead of referencing a card_id, we reference
an action_id.

Emitters may be removed entirely in a future PR.

* Fix extra require lint

* Add checks for superuser and feature enabled to execution route. Fix code review comments

* Add missing comment to migration

Addresses #24951, part of #24861.

* Create collection together with the app
* Don't create an app collection on FE

Co-authored-by: Anton Kulyk <kuliks.anton@gmail.com>

* Add `app_id` property to `Collection` type

* Add `isDataAppCollection` utility

* Use data app icon for app collections

* Simplify navbar tree collection filtering

* Use `import type` syntax

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

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

* Dummy setup

* Wait for the first job to finish

* Use volumes

* Use docker

* Run Cypress

* Initial setup

* Full upgrade test

* Try using maps in a matrix

* Fix breaking string

* Let all matrix workflows finish

* Update `matrix` naming

- `upgrade` is too specific.
- `version` gives us the flexibility to go in both directions (upgrade/downgrade)

* Rename the job to `cross-version-testing`

* DRY setup code

* Enable cross-version testing for EE edition

* Remove test retries

* Pass the secret to the Docker container

* Spice things up

* Schedule a job every Sunday at 9

* bump driver

* prototype of fix

* test

* add note to driver-changelog.md

* Register `users` helpers

* Replace hard coded user mentions in E2E tests

* Add `collections` map to `EntitiesState` type

* Add `DataApp` type to `metabase-types/api`

* Add `createMockDataApp` test helper

* Extract `DEFAULT_COLLECTION_COLOR_ALIAS` property

* Extract collection `name` and `description` fields

* Add `DataAppsApi`

* Add `DataApps` entity

* Add data app URL builder utility

* Sort `NewItemMenu` imports

* Add basic "New App" flow

* Add basic app routes mirroring collections for now

* Show apps in a dedicated sidebar section

* Rename `dataAppInternal` URL utility

* Convert data app entity file to TypeScript

* disallow writable queries outside of actions api

- handles cards, dashboards, embedding and pulses
- tests for disallowing writable query execution
- fix namespace decl

* fix ns decl

* use `false?` to avoid any nil cards causing a 405

* check that the card is there

* Revert "check that the card is there"

This reverts commit 5b56e2d00291bd21c0461a0567537769b5f73a83.

* Revert "use `false?` to avoid any nil cards causing a 405"

This reverts commit 72569d353ee5830f37d1a14c46289afbe511841b.

* [Toucan 2 prep] Don't invoke Toucan models as functions

* Some fixes

* Test fixes

* Test fix

* [Toucan 2 prep] Don't call `type` or `class` on Toucan models

* Test fixes

* More test fixes 

* Replace perms protocol with multimethods; derive models from perms policy keywords

* Test fixes 

* Appease Eastwood

* Fix errors now that App has been merged in

* Empty commit to trigger CI

* Hydrate app_id for collections with apps attached

Addresses #24941, part of #24861

* [E2E] Fix/update `createNestedQuestion` helper

* Handle undefined values

* Slightly speed 14787 up

* Add validation to card PUT and POST endpoints to invalidate saving models with variables

* Fix validation for snippets and saved question CTEs

* Move validation to pre-update and pre-insert instead of API endpoints

* Fix lint warning

* Replace hard-coded id

* Optimize DB calls in card pre-update

* Accept formatting suggestion

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

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

* Remove non-existent route

* Make some internal routes easier to access

* Show data point value on static bar chart

* Show data point values on static line chart

* Show data point values on static area chart

* Remove unused function

* Improve multiple series area chart data point value legibility

* Render stacked area chart data point values

* Automatically format long values with compact format

* Sample data point values on dense chart

* Improve bar chart values legibility