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(experiments): implement reusable metrics #27154

Merged
merged 20 commits into from
Dec 27, 2024

Conversation

jurajmajerik
Copy link
Contributor

@jurajmajerik jurajmajerik commented Dec 25, 2024

Required feature flag: experiments-multiple-metrics

Changes

Add the ability to manage saved metrics for Experiments.

List saved metrics:
image

Edit a Trends metric:
image

Edit a Funnels metric:
image

How did you test this code?

👀

Copy link
Contributor

github-actions bot commented Dec 26, 2024

Size Change: +172 B (+0.02%)

Total Size: 1.11 MB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 1.11 MB +172 B (+0.02%)

compressed-size-action

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@jurajmajerik jurajmajerik marked this pull request as ready for review December 26, 2024 15:01
@jurajmajerik jurajmajerik requested review from neilkakkar and a team December 26, 2024 15:03
Copy link
Collaborator

@neilkakkar neilkakkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

didn't try running locally, though code looks alright! 🙈

One comment, would recommend running through once locally with the flag off.

@@ -211,11 +221,14 @@ export function Experiments(): JSX.Element {
{ key: ExperimentsTabs.Yours, label: 'Your experiments' },
{ key: ExperimentsTabs.Archived, label: 'Archived experiments' },
{ key: ExperimentsTabs.Holdouts, label: 'Holdout groups' },
{ key: ExperimentsTabs.SavedMetrics, label: 'Saved metrics' },
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should the tab show up without the flag?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, will do!

@jurajmajerik jurajmajerik merged commit 6c58e55 into master Dec 27, 2024
99 checks passed
@jurajmajerik jurajmajerik deleted the experiments-saved-metrics branch December 27, 2024 12:31
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.

3 participants