* fix: use cached insight if the scene has it
* add a cypress test
* better empty checking
* Update UI snapshots for `chromium` (2)
* Update UI snapshots for `chromium` (2)
* slightly less bad
---------
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* feat: a little more e2e cypress data exploration tests
* Update UI snapshots for `chromium` (2)
* Update UI snapshots for `chromium` (2)
* tests were passing by accident
---------
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* feat: allow new insight type choice from sidebar
* update e2e test
* add more e2e test fixes
* Update frontend/src/layout/navigation/SideBar/SideBar.tsx
Co-authored-by: Michael Matloka <dev@twixes.com>
* a plus and a width
* allow passing in max width to popover
* Revert "allow passing in max width to popover"
This reverts commit 415a751f71.
* third-ish times the charm
* Use a more concise tooltip
The tooltip basically functions as a on-hover label here, so using the same text as on the saved insights page, where it says "New insight".
* Update UI snapshots for `webkit` (2)
* Update UI snapshots for `webkit` (2)
---------
Co-authored-by: Michael Matloka <dev@twixes.com>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Problem
We're two major versions of Cypress behind. They claim the newer versions are faster
Changes
dependency updates
a few fixed tests for new syntax
* chore(deps): Update posthog-js to 1.42.2
* revert test
Co-authored-by: posthog-bot <posthog-bot@users.noreply.github.com>
Co-authored-by: eric <eeoneric@gmail.com>
* Get rid of control version
* Don't show option to license on billing page
* Remove ff constant
* Include plans in billing response
* Update types for the plans
* Hook up the top of the table
* Fill out the rest of the table
* Hook up the rest of the table
* Add tooltip with feature descriptions
* Fix type errors
* Fix more type issues
* Remove old plan data
* clean up
* exclude features we don't want to show in table
* Tolerate missing plan keys
* fix mypy
* Update cypresss test
* Fix mypy again
* Just get whatever default plans
* fix tests
* Update snapshots
* Update snapshots
* moar test fixes
* use is_free to know if free plan
* Say the free allotment is incl'd if base has $ price
* Update snapshots
* Don't change anything having to do with licenses
* Fix type error
* Update snapshots
* Update snapshots
* Use product name from stripe
* only lowercase
* don't hardcode plan names
Co-authored-by: Emanuele Capparelli <k@emkpp.com>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Problem
@annikaschmid accidentally deleted an insight from a dashboard and then couldn't put it back.
The insight API response was returning the deleted dashboard in the insight's dashboards field. That meant the UI didn't offer the ability to add it to that dashboard because it thought it was already added
Even though there was a test that protected against this which passes and I couldn't figure out why the application behaves well in the test and not when running 🤷
Changes
The dashboard_tiles relation is correct. And we should eventually remove the use of the dashboards relation.
So, let's expose it. This begins the deprecation of the dashboards field on the Insight API.
And overwrites the content of the dashboards field with the correct values when returning a representation of the model.
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Michael Matloka <dev@twixes.com>
## Problem
In this thread https://posthog.slack.com/archives/C0368RPHLQH/p1672383272168819 @macobo spotted that we are loading the list of insights more frequently than we need to due to a code path that keeps the turbo mode cache up-to-date for `savedInsightsLogic`
## Changes
Since the `DashboardsModel` action being reacted to passes the `InsightModel` that was updated. We can update the state in-memory without an API call
## How did you test this code?
added developer tests and checked the site running locally
* Make the product benefits test the default
and put hog on login
* Refactor BridgePage to be just a container
And put extra stuff in the relevant components
* Only let someone set the hog message if hog exists
* Fix test
* Remove the flag constant
* Update SignupContainer.tsx
Co-authored-by: Emanuele Capparelli <kappa90@users.noreply.github.com>
Problem
Via customer feedback: If someone is duplicating a dashboard they might not want to duplicate the insights on it.
Changes
This lets them choose.
