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

feat(data-warehouse): Add a billable field to jobs #27435

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Gilbert09
Copy link
Member

Problem

  • To prep for the V2 pipeline release, we need a better way to distinguish which jobs the customer should be billed for and what jobs should appear in the posthog UI

Changes

  • Added a billable boolean field to the jobs model
    • Defaults to true
  • Added a migration to set all jobs as billable except for the V2 jobs
  • When we release the pipeline, we just set the V2 jobs as billable and V1 jobs as not billable, no other changes needed in terms of billing and jobs in the UI

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Updates tests

@Gilbert09 Gilbert09 requested a review from EDsCODE January 10, 2025 12:45
@Gilbert09 Gilbert09 enabled auto-merge (squash) January 10, 2025 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant