Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Static cohorts + upload by CSV #2868

Closed
timgl opened this issue Jan 6, 2021 · 1 comment
Closed

Static cohorts + upload by CSV #2868

timgl opened this issue Jan 6, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@timgl
Copy link
Collaborator

timgl commented Jan 6, 2021

Is your feature request related to a problem?

I have a list of user ids (distinct IDs in PostHog) in CSV format that I want to upload as a static cohort so I can track that set of users.

Describe the solution you'd like

When creating a cohort, give me the option to upload a CSV instead of defining it the normal way. This static cohort should have a label static in the list, and should not be automatically updated.

Describe alternatives you've considered

Additional context

Thank you for your feature request – we love each and every one!

@timgl timgl added the enhancement New feature or request label Jan 6, 2021
@paolodamico
Copy link
Contributor

I think we should also support just pasting the URLs from clipboard (from a spreadsheet or something). For most cases, I think this will be faster and more intuitive. You would probably do CSV when you're importing a ton of users.

@EDsCODE EDsCODE mentioned this issue Jan 8, 2021
21 tasks
timgl added a commit that referenced this issue Jan 13, 2021
@timgl timgl closed this as completed in 1a6207d Jan 15, 2021
EDsCODE added a commit that referenced this issue Feb 24, 2021
* Upload static cohort using CSV, closes #2868

* fix tests

* Fix tests

* Fix e2e test

* Avoid double inserts

* Speed up query

* Move to params

* fix tests

* initial cleanup

* initial scaffold for cohort static logic

* intiial frontend changes

* initial working

* fix error on iterable

* stickiness people func refactor

* Use JSON instead of protobuf

* stickiness working

* clickhouse route

* unused import

* move button

* make interface easier to user

* clickhouse support

* restore cache decorator

* endpoint adjustment

* adjust how clickhouse handles inserting for static cohorts

* split out processing into pg

* use worker for clickhouse querying too

* add pg test

* ee test

* add input for cohort naming

* add stickiness test

* condition for when to show save cohort

* fix types

* minor fix

* remove repeats

* remove hardcoded id

* add date handling to trends cohort func

* remove print

* add link on success creation

* cohort working

* fix backend test

* fix cohort typescript

* remove unnecessary kafka topic

* remove brackets

* shift default

* raise exceptions when debug or test is true

* remove test flag

* move trends insight related components to different folder

* change get_earliest_timestamp

* remove extraneous condition

* fix test call

* put behind ff

* fix test x2

* fix tests

* restore exceptions hog

* fix tests

* restore exceptions hog

* fix test x20

Co-authored-by: Tim Glaser <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants