-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix(demo): allow regular org members to create demo proj #13582
Conversation
subtext="Explore insights, create dashboards, try out cohorts, and more." | ||
/> | ||
) : null} | ||
<DemoProjectButton |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be behind the new flag you created?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I put the check inside that component since it was implemented in a couple different places. https://github.com/PostHog/posthog/pull/13582/files#diff-b45ab9c675aae471e2211c373c0d3bbdb6faaf7f6db73d1a9e72594863f1744eR74
)" This reverts commit 0c1ff71.
)" This reverts commit 0c1ff71.
Problem
Only admins were allowed to create demo projects. Should allow anyone in the org to do so.
Changes
This first reverts the revert in da3af1c. So you only need to review changes from then on.
👉 Stay up-to-date with PostHog coding conventions for a smoother review.
How did you test this code?
Added a test to make sure regular members can create a demo team