* 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>
* feat(toolbar): rename editor to toolbar, add toast on error
* add missing types
* be less noisy when expired token comes a previous page load
Co-authored-by: Michael Matloka <dev@twixes.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.
Problem
Via customer feedback: If someone is deleting a dashboard they want to delete the insights too. At the moment that means they have to delete them one at a time.
Changes
Introduces a modal when deleting a dashboard that lets you choose whether to delete the insights on the dashboard
Soft-deletes the insights as appropriate
* revert: #11552
* feat: save title and description on blur
* allow undo when renaming insight
* minimise change
* minimise change
* fix cancelling and add conditional updatedaashboard undo
* no need to change saved insights logic
* only save on blur if value has changed else cancel
* add cypress test to rename insight
* add a cypress test that you can change (and revert that change) an insight name
* wait for insight save to complete
* use lemon components for taxonomic property filters
* add property value class
* fix
* use lemon select props instead
* slightly less generic classname
* add back classname for tests
* do not use muted alt
* label with classname
* feat: Ungrid filter row to help with lemonizing (#11733)
Co-authored-by: Ben White <ben@benjackwhite.co.uk>
* fix: remove data tooltip and improve prompts rules
* fix test fixtures
* simplify expression
* fix cypress
* add cache prefix
* Update frontend/src/lib/components/LemonButton/More.tsx
Co-authored-by: Marius Andra <marius.andra@gmail.com>
Co-authored-by: Marius Andra <marius.andra@gmail.com>
* feat(data-management): add custom events list
* remove dead code
* fix test
* assert what matters
* this seems flakey, even locally, though the interface shows the right data locally... testing a timeout
* new script
* fix test
* remove frontend changes (PR incoming)
* describe meaning behind symbols
