-
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
feat(feature flags): add endpoint for copying a flag to multiple projects #18421
Conversation
Co-authored-by: Neil Kakkar <[email protected]>
…g/posthog into fix/feature-flags-filters
…feature-flags-filters
…g/posthog into fix/feature-flags-filters
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.
so far so good!
posthog/api/test/__snapshots__/test_organization_feature_flag.ambr
Outdated
Show resolved
Hide resolved
(sorry if it seems I'm being too pedantic, but with create-update operations that automatically happen, really need to make sure things are solid and all code paths are tested so this doesn't come back to bite us) |
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.
great!
"usage_dashboard": "__ignore__", | ||
"is_simple_flag": True, | ||
"experiment_set": [], | ||
"surveys": [], |
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.
in a follow up, add survey/experiment/early-access features linked to this flag, and ensure they aren't overridden :)
Changes
Add an endpoint for copying a flag to multiple projects.
POST /api/organizations/:org_id/feature_flags/copy_flags
Ability to copy multiple flags at once will follow.
How did you test this code?
API tests