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

chore(flags): clarify return values for boolean flags when no release condition matches #10477

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

Conversation

havenbarnes
Copy link
Contributor

Changes

Context: posthog.slack.com/archives/C07Q2U4BH4L/p1737751112214639

Tweaking description of boolean flags to match actual API/SDK behavior

Related change to feature flag usage tab: PostHog/posthog#27888

Copy link

vercel bot commented Jan 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
posthog 🛑 Canceled (Inspect) Jan 24, 2025 10:24pm

@@ -84,8 +84,6 @@ A payload is an additional piece of information sent to your app when a flag is

They enable you to configure functionality related to your flag inside PostHog, instead of having to make code changes or redeploy your app.

> **Note**: Payloads are only available in our JavaScript web, Node, Python, Ruby, and React libraries.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Drive-by fact check. This is false, any SDK that supports flags supports payloads

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