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

management/commands errors #3060

Closed
sophie200 opened this issue Jan 23, 2021 · 1 comment · Fixed by #3061
Closed

management/commands errors #3060

sophie200 opened this issue Jan 23, 2021 · 1 comment · Fixed by #3061
Labels
bug Something isn't working right

Comments

@sophie200
Copy link

Bug description

(Import lines)
https://github.com/PostHog/posthog/blob/master/posthog/management/commands/create_bulk_events.py#L21
https://github.com/PostHog/posthog/blob/master/posthog/management/commands/create_bulk_events.py#L22

An ImportError is raised when you run this module because Funnel and FunnelStep have been deleted. There are also some queries that reference these deleted models as well. I was wondering does it matter if this module is updated because it seems like this file hasn't been updated in a while, or are there other reasons that these queries and imports still exist? Thanks.

(Lines of some queries that reference deleted models)
https://github.com/PostHog/posthog/blob/master/posthog/management/commands/create_bulk_events.py#L220
https://github.com/PostHog/posthog/blob/master/posthog/management/commands/create_bulk_events.py#L221

Environment

self-hosted PostHog, master branch (11/21/2021)

@sophie200 sophie200 added the bug Something isn't working right label Jan 23, 2021
@yakkomajuri
Copy link
Contributor

Oh yeah thanks for the heads up on this! We don't really use this command anymore so I'll just go ahead and remove it. In the meanwhile you can use generate_revenue_data or take a look at this PR: #2629

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working right
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants