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: record non-onboarding product intents for data warehouse #25859

Merged
merged 9 commits into from
Oct 29, 2024

Conversation

raquelmsmith
Copy link
Member

@raquelmsmith raquelmsmith commented Oct 28, 2024

Problem

Data warehouse isn't a product we expect most people to start with. So, we should evaluate if interested people are getting set up with it properly using some other starting point.

Changes

Record a product intent when someone clicks the Link button on a source.

image

@PostHog/team-data-warehouse is this something you think reasonably shows interest in the product aside from just clicking around? We can also record it after someone clicks a couple times, or does some other action.

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

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

How did you test this code?

Need to update tests.

@raquelmsmith raquelmsmith requested a review from a team October 28, 2024 20:13
@raquelmsmith raquelmsmith enabled auto-merge (squash) October 29, 2024 02:50
Copy link
Contributor

github-actions bot commented Oct 29, 2024

Size Change: +22 B (0%)

Total Size: 1.15 MB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 1.15 MB +22 B (0%)

compressed-size-action

@raquelmsmith raquelmsmith disabled auto-merge October 29, 2024 16:07
@raquelmsmith raquelmsmith enabled auto-merge (squash) October 29, 2024 16:11
@raquelmsmith raquelmsmith merged commit e6877d3 into master Oct 29, 2024
92 of 93 checks passed
@raquelmsmith raquelmsmith deleted the feat/product-intent-dw branch October 29, 2024 16:52
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.

2 participants