0
0
mirror of https://github.com/PostHog/posthog.git synced 2024-12-01 12:21:02 +01:00
Commit Graph

1911 Commits

Author SHA1 Message Date
Marius Andra
b720da94ab
Saved insights filter & url sync (#6388)
* saved insights url sync WIP

* sync filters with the url

* improve trend and date filters

* convert utils test to typescript

* create simple object shallow diff function

* only persist the difference from the default filters in the url

* add tests and simplify operations

* improve tests

* better load checking
2021-10-22 11:53:40 +02:00
Rick Marron
3f8eaeb843
Fix error in recordings table (#6598)
* fix

* Make person header seem more click-able

* clean up
2021-10-21 18:58:28 +00:00
Jaspreet singh
10a4a6a662
Closes 6541 add step to empty funnel (#6556)
* Closes 6541 add step to empty funnel

* added comments back

* hide delete button when single step exists

* use values directly in first step

* Update frontend/src/scenes/insights/ActionFilter/ActionFilterRow/ActionFilterRow.tsx

Co-authored-by: Neil Kakkar <neilkakkar@gmail.com>
2021-10-21 16:44:32 +00:00
Neil Kakkar
716e8f1943
Open modals for everything in correlation events (#6592)
* Open modals for everything in correlation events

* address comment
2021-10-21 14:57:27 +00:00
Paul D'Ambra
9e552d51e4
Search toolbar actions (#6587)
* add search to toolbar actions display for #6202

* better markup and spacing

* removes a console.log that snuck in
2021-10-21 10:37:14 +02:00
Harry Waye
e4939b767e
chore(eslint): explicitly disable exhaustive deps on historical code (#6576)
I haven't made any attept here to resolve anything here, just to
explicitly acknowledge that these are issues that eslint has highlighted
and we should ignore, for the sake of easily picking up new issues. If
these are truely problematic, then I am assuming they should have been
picked up as bugs with the system.

This doesn't stop someone from coming in and reviewing the code in this
pr retrospectively.
2021-10-21 09:35:34 +01:00
Eric Duong
6160d0c30e
add properties to path cleanfilters (#6554) 2021-10-21 10:29:01 +02:00
Paul D'Ambra
a957f0abb0
adds a search box to the toolbar featureflag list (#6527)
* adds a search box to the toolbar featureflag list

* test flakes because react is re-rendering between the get and the click. add an assertion to try and slow cypress down to avoid this

* move filtering toolbar feature flags to a Set and out of CSS

* support initKeaTestLogic() with no args

* fix query-selector-all-deep jest bug

* add simple test case for feature flags logic

* combine selectors

* with more understanding of Fuse

* add simple test for flag filtering

Co-authored-by: Marius Andra <marius.andra@gmail.com>
2021-10-20 22:02:02 +00:00
Marius Andra
f84b447fce
Include the insight ID in the url for "saved funnels" (#6531) 2021-10-20 14:17:40 -07:00
Paolo D'Amico
4fd848d4d3
Debug queries (#6577) 2021-10-20 14:17:30 -07:00
Michael Matloka
4a7d0b7a7e
Use app context in organizationLogic too (#6565)
* Use app context in `organizationLogic` too

* Add `organizationLogic.test.ts`
2021-10-20 18:46:58 +02:00
Harry Waye
01873b1780
feat(correlation): add multiselect for property names (#6407)
* refactor(taxonomic-filter): add api mock functions

* chore: remove makeApi from storybook preview.js

* feat(correlation): add multiselect for property names

Previously we had a simple text input box that was splitting the input
to get a list of property names. This doesn't give the greatest user
experience.

Now we have a simple select box that includes:

 1. search filtering for options
 2. multiselecting options
 3. a select $all, although this is pretty poorly implemented at the
    moment, for instance you can select $all, at the same time as
    selecting a subset of properties.

* add a little commentary around PropertyNamesSelect

* add deps to useEffect

* ignore return type eslint issues

* just use console.log instead of actions

* remove unused import

* chore: add error condition

* chore(correlation): update styling of property select

This puts it more in line with
https://www.figma.com/file/gQBj9YnNgD8YW4nBwCVLZf/PostHog-App?node-id=4200%3A30715

* Add no search results message

* input focus border

* fix type

* use onBlur

* make checkbox checked colour change

* make query bold in no results message

* add clear

* Implement search highlighting

* fix typing

* add regex split comment

* if no property names selected, default to $all

* typo

* Add test for selection component

* Add test highlighting onChange lag and clicking outside

* click out side should depend on hide

* make sure onChange triggered on clear or select all
2021-10-20 14:12:32 +00:00
Paolo D'Amico
b326eba355
Hide conversion rate on trends (#6561) 2021-10-20 11:37:57 +01:00
Paul D'Ambra
1cb0ba2bbd
Simplified UI for breakdown properties (#6557)
* wip implementation of grey pills for breakdown properties

* cohort pills need coercing differently to line up with the close icon

* which allows removal of some markup

* much better layout and matching behaviour

* resolve type errors in trends tab

* resolve type errors on funnels tab

* remove console.logs
2021-10-20 09:48:42 +01:00
Alex Gyujin Kim
ba33121d33
Move rrweb player into repo (#6505)
* rename sessionRecording to session-recording

* logic for new player

* move rrweb-player into repo

* remove awkward wrapper and fix time not updating bug

* chunk load meta timing

* fix duration setting bug

* update icons and styling for controller

* styling icons

* slider styling

* ugly slider

* remove consoles

* sonarqube

* typescript

* typescript

* remove hardcode

* Fix icon from merge

* move report usage back to sessionsplaylogic

* add comments back

* bug fix

Co-authored-by: Rick Marron <rcmarron@gmail.com>
2021-10-19 21:33:43 -07:00
Michael Matloka
9b54eb7c41
Pilot migration for logics to be project-based (#6430)
* Instrument legacy endpoint debugging

* Add `currentTeamId` to `teamLogic`

* Add logics utils

* Update annotations endpoint in the frontend

* Limit legacy endpoint logging to DEBUG

* Fix `annotationsTableLogic` usage

* Update test_annotation.py

* Add `test_deleting_annotation_of_other_team_prevented`

* Comment out debug code

* Rework approach

* Remove redundant lines

* Fix mixup

* Fix non-logged-in scenes

* Fix Python cast
2021-10-20 04:55:39 +02:00
Paul D'Ambra
f472b163e2
adds a feature flag constant for #938 (#6555) 2021-10-19 16:32:30 +00:00
Yuvaraj J
f34bd5551f
Allow duplicating dashboard (#6476)
* Closes #1688 Allow dashboards to be duplicated in the backend

* Allow dashboards to be duplicated in the frontend

* Remove unused method

* Filter existing dashboard along with team id

* Add title attributes to actions in dashboard listing page
2021-10-19 18:10:27 +02:00
Neil Kakkar
8edd14a16a
Create a very basic property correlations over event table (#6549)
* Create basic nested tables for correlation properties results

* use correct API responses

* clean up

* fix typing

* try 2
2021-10-19 16:20:11 +01:00
Yakko Majuri
500cf5bc7f
export from the beginning -> export historical events (#6542) 2021-10-19 12:55:19 +00:00
Tim Glaser
6be8918c12
fix cohort static select (#6519)
* fix cohort static select

* Remove unused button
2021-10-19 14:27:33 +02:00
Paul D'Ambra
23cf2e9280
Remove expensive person header logic allocation (#6532)
* remove expensive person header logic allocation

* rename the person header test
2021-10-19 10:23:18 +00:00
Paolo D'Amico
c587fa922d
Correlation - Skew banner (#6511)
* fix override on debug

* new skew warning
2021-10-19 09:12:28 +02:00
Alex Gyujin Kim
192c2e293c
time conversion zero (#6521) 2021-10-19 09:04:51 +02:00
Michael Matloka
faa1323960
Proposed cleanup of events table column configurator (#6523)
* Proposed cleanup of events table column configurator

* Fix modal footer spacing
2021-10-18 21:44:55 +02:00
Paolo D'Amico
ff74653af2
No history on saved insights (#6510) 2021-10-18 20:15:28 +03:00
Michael Matloka
23195788e4
Improve wording of ErrorProjectUnavailable (#6516) 2021-10-18 17:26:00 +02:00
Paul D'Ambra
7f4fa88fa0
corrects a sentence (#6515) 2021-10-18 16:03:16 +01:00
Marius Andra
470090b865
Fix "current usage" padding (#6501) 2021-10-18 11:39:30 +02:00
Marius Andra
40cbfddf27
Dashboard initial load in batches of 4 (#6494)
* add doNotLoad to insights on dashboards that come without results

* load dashboard items without results in sequence

* fix test

* don't use the visibility map if it's empty (no keys)

* Update frontend/src/scenes/dashboard/dashboardLogic.tsx

Co-authored-by: Paolo D'Amico <paolodamico@users.noreply.github.com>

* work with both dashboards and saved insight card views

Co-authored-by: Paolo D'Amico <paolodamico@users.noreply.github.com>
2021-10-18 07:44:55 +00:00
Paolo D'Amico
45f954548b
Available features selector (#6506)
* hasAvailableFeature selector

* use selector everywhere
2021-10-18 07:43:03 +00:00
Paolo D'Amico
4248d52800
Paths pay gate (#6463)
* create new advanced feature

* remove all advanced features

* pay card basic

* dismiss pay gate

* pay card story component

* address feedback

* restricted paths step too
2021-10-18 09:17:54 +02:00
Paolo D'Amico
39ff3abab2
Persisted flags for 1.29.0 (#6401) 2021-10-18 02:32:20 +03:00
Paolo D'Amico
faa6377a57
Login minor UI fixes (#6472)
* fix spacing issue in signup page

* fix spacing with SSO links

* uncomment temp code
2021-10-15 22:11:16 +00:00
Michael Matloka
4477c81f1f
Optimize dashboardsModel a tiny bit (#6455)
* Optimize `dashboardsModel` a tiny bit

* Fix typing

* Make sorting more reliable

* Use more correct `dashboardModel` selectors, with better names

* Fix value name

* Fix and simplify `nextDashboard`

* Rename selectors
2021-10-16 00:07:45 +02:00
Alex Gyujin Kim
70653001f8
New component for new sessions player (#6486)
* new ff for new sessions player

* put into session recordings folder

* add kea typing
2021-10-15 12:39:24 -07:00
Paolo D'Amico
4b7ea0f5cc
Fix password reset (#6466)
* fix render issue on email

* fix redirection after password change

* fix test
2021-10-15 15:58:54 +02:00
Marius Andra
85aa9f3b74
Fix Cypress component tests (#6461)
* fix component tests

* disable sessions test

* add console errors into cypress output
2021-10-15 12:47:06 +02:00
Marius Andra
296647fa10
Override filters if bad api result (#6470)
* dispatch loadResultsFailure if failed to load

* add tests for API failures

* don't mount dashboards model if it's not mounted
2021-10-15 12:45:52 +02:00
Marius Andra
3a9db09a5f
Fix dashboard and saved insight reload (#6456)
* reload insights on dashboards, clean up utils

* refresh items

* fix test

* test reloading with a fixed list

* show "0 out of N" where N is 1 or all

* add comment re: custom reload logic
2021-10-15 08:53:09 +00:00
Marius Andra
2b571d0c8e
Fix event table re-render loop if person == null (#6460)
* fix person == null re-render loop

* fix test
2021-10-15 10:21:12 +02:00
Paolo D'Amico
20b0f9704a
Forgot password UI (#6177) 2021-10-15 10:40:31 +03:00
Paolo D'Amico
68f79ba04a
Release 5720-plugins-ui-jobs (#6438) 2021-10-15 10:17:55 +03:00
Paolo D'Amico
b0309c2390
Release 4871-plugin-metrics FF (#6437) 2021-10-15 10:17:49 +03:00
Marius Andra
3d3a6594c6
Hotfix for from_dashboard filter bug (#6450)
* hotfix for from_dashboard filter bug

* catch if totally empty

* fix feedback

* improve error
2021-10-15 09:16:48 +02:00
Eric Duong
2fbe638f40
Dashboard item: Only pass object id when deleting dashboard item (#6447)
* only pass object id

* add loadresults

* add tests that validate loadResults exists on insight logics for dashboard item refresh

* add name to payload

* remove refresh fix
2021-10-15 09:10:30 +02:00
Paul D'Ambra
35efae5615
treat minutely and hourly graphs differently when rendering annotations (#6448)
Co-authored-by: Marius Andra <marius.andra@gmail.com>
2021-10-15 08:56:57 +02:00
Paolo D'Amico
7e56333c56
Password reset - API (#6335)
Co-authored-by: Michael Matloka <dev@twixes.com>
2021-10-15 09:33:03 +03:00
Michael Matloka
f5b2f36f29
Fix duplicated action name error message (#6454)
* Fix import of `urls` in `ActionEdit`

* Fix duplicated action name error message
2021-10-15 08:18:15 +02:00
Rick Marron
a5405b7a65
check for new values before setting filters (#6457) 2021-10-14 23:09:52 -07:00