* adding border to pivot table sort buttons

* Changing to use Button component

* treat private-key conn props as base64 encoded

- fix typo

* handle connecting with local keypair

* adds test for snowflake keypair auth

* remove unused arg

* remove id, and only call byte-array->string once

* dont reflect for conversion from byte-array to str

* remove inline def

* more cleanup

- mess with tests to find the fabled metabase ci snowflake user

* remove un-needed test

* handle conn uri in connection-details->spec

* make sure tests are actually running

* encode properties in the connection uri

* typehint private keyfile

* respond to some review comments

- add a test for can-connect? => true, even when jdbc/query returns falsey

* remove unused logging require

* adds assoc-qp

- which will be used to non-destructively "assoc" onto the snowflake
connection query params

* hook in u.qp/assoc-qp to better update conn uri

* kill a newline

* reuse `conn-str-with-additional-opts` on snowflake conn uri

* update use utf-8 when encoding private-key-value

conveniently use `bytes` and `bytes?`

* Updating Eye Icons

* Updating Tests

* Fix weird static viz data point values rounding

* Fix incorrect static viz formatting

* Fix failing tests

* Ignore only FE unit test changes in backend workflows

* Ignore both FE and E2E tests

* Update models.md

* Update docs/troubleshooting-guide/models.md

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

* Update models.md

* Update models.md

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

* Extract `DashCardVisualization`

* Fix click behavior

* Fix `dispatch` prop

* Revert "Fix incorrect static viz formatting (#26737)"

This reverts commit c155666c.

* Revert "Fix weird static viz data point values rounding (#26704)"

This reverts commit 25329609.

* return white text color more frequently

* increase factor

* Extract action button components

* Extract `DashCardActionButtonsContainer`

* Add shared `DashCardActionButton` component

* Clean up `AddSeriesButton`

* Clean up `ChartSettingsButton`

* Clean up `ToggleCardPreviewButton`

* Clean up `RemoveButton`

* Clean up click behavior button

* Make remove and toggle preview buttons inline

* Clean up `DashCardActionButtons`

* Fix type error

* Adding graph.series_order_dimension

* Handling adding and removing order items

* Adding test for funnel viz

* PR Feedback

* Simplifying ChartSettingOrderedSimple. Moving logic to Settings Defs

* Fixing Row Chart rendering, adding bar chart test

* Resolving type check errors

* Adjusting series_order empty values

* Updating orderItem prop check

* adjusting getOrderedSeries

* Extract styled components

* Convert to TypeScript

* Fix styling

* Fix type error

* Add `query_average_duration` to `Card` type

* Add missing props to `DashboardOrderedCard` type

* Use up-to-date Card type for DashboardOrderedCard

* Extend `Series` type

* Convert `DashCard` to TypeScript

* Move hidden background style to styled component

* Clean up props a little bit

* Extract methods

* Sort props and extract rendering methods

* Extract getter methods

* Turn `DashCard` into a functional component

* Extract styled components

* Fix size properties

* Sort imports

* Move styled component to `.styled` file

* Extract `DashCardActionButtons`

* Extract `ClickBehaviorSidebarOverlay`

* Move `DashCard` exports to the bottom

* Extract `propTypes` from class definition

* Use initial state shortcut declaration

* Update running-metabase-on-azure.md

Having an `@` symbol in the connection string may cause issues when it is parsed and rebuilt.  Using separate user and password configuration values solves that problem.

* Update docs/installation-and-operation/running-metabase-on-azure.md

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

* Move `DashCard` to its own directory

* Move `DashCardParameterMapper` to `DashCard` dir

* Remove disabled `DashCard` test files

* Fix import

* Move around `DashCardParameterMapper`

* fix row chart bugs

* review

* make it clearer you're creating a new question

* fix the function

* Fix bug where grouping strategy would NPE when fingerprint was `nil`

The axes grouping relies on fingerprint data existing, AND for now only works on :type/Number columns, so this bugfix
guards against those situations where the logic would fail.

The bug that brought this to my attention came up when there was a Custom column. This set the col-meta count to be
one greater than the count of the 'real' columns, which threw off my old assumptions, where if the grouped-columns
count was less than the total col-meta count, you had a potential grouping. This broke in part because some custom
columns are essentially derived from a real column, and so will most likely be groupable with the column they're
derived from.

Anyway, this filters out columns that won't work, so we hopefully eliminate the problem.

* Update src/metabase/pulse/render/body.clj

Co-authored-by: Tim Macdonald <tim@metabase.com>

* Fix some broken logic

Co-authored-by: Tim Macdonald <tim@metabase.com>

* Move collection entity tests to entity directory

* Simplify `normalizedCollection` helper

* Remove nested ternaries in `getCollectionType`

* Simplify list endpoint code

* Remove comment

* Simplify null check

* Fix collection's `authority_level` types

* Convert collection utils to TypeScript

* Convert `index` file to TypeScript

* Move `getInitialCollectionId` to its own file

* Move `getExpandedCollectionsById` to its own file

* Simplify `getExpandedCollectionsById` usage

* Convert collections entity to TypeScript

* Add missing type

* Fix type errors

* Remove embarrassing backticks