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

consolidate both, change sidebars #2435

Merged
merged 2 commits into from
Dec 24, 2024
Merged

Conversation

brock-statsig
Copy link
Contributor

No description provided.

@brock-statsig brock-statsig enabled auto-merge (squash) December 24, 2024 01:13

If you want to pass a userID for the above reasons, but don't have a logged in user (e.g. you are optimizing the login flow), set a stable identifier as a cookie or in local storage and use that with each call to Statsig.
1. If you plan to only use one/off feature gates, or non-percent-based rules (like countries)
While you're still losing functionality, you can **pass any non-empty identifier, hard coded string, or **a random ID less than 100 if you do not have the actual user ID.** Don't pass a purely random ID - as we won't be able to dedupe your events, you'll explode your event usage, and your Statsig bill.
Copy link

Choose a reason for hiding this comment

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

There's a markdown formatting issue with double asterisks in this line. The correct formatting should be:

While you're still losing functionality, you can **pass any non-empty identifier, hard coded string, or a random ID less than 100 if you do not have the actual user ID.** Don't pass...

This will properly render the bold text without breaking the markdown.

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.


If you want to pass a userID for the above reasons, but don't have a logged in user (e.g. you are optimizing the login flow), set a stable identifier as a cookie or in local storage and use that with each call to Statsig.
1. If you plan to only use one/off feature gates, or non-percent-based rules (like countries)
Copy link

Choose a reason for hiding this comment

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

Minor typo: one/off should be hyphenated as one-off

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.

@jasonwzm
Copy link
Member

jasonwzm commented Dec 24, 2024

@jasonwzm jasonwzm temporarily deployed to pull request December 24, 2024 01:15 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request December 24, 2024 01:29 Inactive
@brock-statsig brock-statsig merged commit ebab7d6 into main Dec 24, 2024
3 checks passed
@brock-statsig brock-statsig deleted the consolidate-user-object-pages branch December 24, 2024 01:29
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