* Fix incorrect question variable links

* Extract method building question URL

* Unskip the repro

* Add `isPersonalCollectionChild` utility

* Hide permission control for personal collections

* Hide permissions modal for personal collections

* Handle missing `collection.effective_ancestors`

* Use `sidebar` cypress selector

* fix trend visualization is not visible if the first column is not date

* skip repro

* review fix

* don't apply parameter filters to object queries

* unskip #8055 repro as well

* add units to new columns, add hover state for column headers

* fixup! add units to new columns, add hover state for column headers

* hide loader when filtering or sorting audit table

* fix tests

No tests caught this because of compounding errors on top of a generic
solution that already worked:

for h2, defined unix-timestamp->honeysql on [:h2 :millisecond] but
:millisecond is not a valid dispatch key, it is always
milliseconds (plural). And it was duplicated forgetting to use
:microseconds.

But this all worked because there are defaults in the
sql/query_processor

```clojure
(defmethod unix-timestamp->honeysql [:sql :milliseconds]
  [driver _ expr]
  (unix-timestamp->honeysql driver :seconds (hx// expr 1000)))

(defmethod unix-timestamp->honeysql [:sql :microseconds]
  [driver _ expr]
  (unix-timestamp->honeysql driver :seconds (hx// expr 1000000)))

```

so h2 was always using these fallbacks with the division:

```sql
"SELECT \"PUBLIC\".\"INCIDENTS\".\"ID\" AS \"ID\",
        \"PUBLIC\".\"INCIDENTS\".\"SEVERITY\" AS \"SEVERITY\",
        timestampadd('second', (\"PUBLIC\".\"INCIDENTS\".\"TIMESTAMP\"/1000000),--note division
                     timestamp '1970-01-01T00:00:00Z') AS \"TIMESTAMP\"
 FROM \"PUBLIC\".\"INCIDENTS\" LIMIT 1048575"
 ```

rather than the h2 optimized form:

```sql
"SELECT \"PUBLIC\".\"INCIDENTS\".\"ID\" AS \"ID\",
        \"PUBLIC\".\"INCIDENTS\".\"SEVERITY\" AS \"SEVERITY\",
        timestampadd('microsecond', --note microseconds
                     \"PUBLIC\".\"INCIDENTS\".\"TIMESTAMP\",
                     timestamp '1970-01-01T00:00:00Z') AS
                     \"TIMESTAMP\"
FROM \"PUBLIC\".\"INCIDENTS\" LIMIT 1048575"
```

The end result is that the defmethods were never called and the
default was used. So a bug, but not one that was visible. Good catch
jeff

Merge `39.4` into `master`

Merge `release-x.39.x` including ae6e9d9f without squash merging

* Move test inside a describe block

* Add repros for #16379

* keep click_behavior settings when reset cards settings

* unskip repro

* review fix

* add seriesIndex back to clicked object

* use transformed series when handled viz click

* unskip repro for related issue

* add additional test for this bug

Fix typo in DS name for app DB

Add test assertion

* Add new 'binning' e2e group to the CI

* Add initial set of tests for the binning options (buckets)

* stop hiding broken params

* Refactor and unskip the repro for #15279 (#16592)

Co-authored-by: Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>

* refetch card on RELOAD_CARD action

* fetch via question entity to update cache

* unskip & fix cy test

* Extract largish sort-order function

also, running tests locally could run into timezone issues. only
interested in comparisons (<) so just make sure that its 24 hours in
the past so no issues with time zones

* tests are pretty literal but just satisfying the test checker

* bump coverage from 71 to hopefully 75%

* Add repro for #16555

* Extract common logic into variables

* Update the assertion

* exclude parameters from isDirty check

* prevent calling methods on null originalQuestion

* rmv Save reliance in e2e test

* code refactor

* unskip repro

* rmv #520 workaround test because bug is fixed

* revert to previous sandbox.cy.spec.js

* skip workaround test

* add explanatory comment