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

add init docs together #2445

Merged
merged 1 commit into from
Dec 27, 2024
Merged

add init docs together #2445

merged 1 commit into from
Dec 27, 2024

Conversation

brock-statsig
Copy link
Contributor

No description provided.

@brock-statsig brock-statsig enabled auto-merge (squash) December 27, 2024 00:03
3. The Server retrieves updated rules from the server, and is now ready for checks even if it didn't find values in step 1.
4. Going forward, the server will retrieve new values every 10 seconds from the server, updating the locally cached values each time.

DataAdapters provide a layer of resilience, and ensure your server is ready to server requests as soon as it startups rather than waiting for a network roundtrip, which can be especially valuable if you have short-lived or serverless instances. While only recommended for advanced setups, Statsig also offers a [Forward Proxy](/server/concepts/forward_proxy/) that can add an extra layer of resilience to your server setup.
Copy link

Choose a reason for hiding this comment

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

Found a few typos in this sentence:

  • "server requests" should be "serve requests"
  • "startups" should be "starts up"

The corrected sentence would read: "...ensure your server is ready to serve requests as soon as it starts up..."

Spotted by Graphite Reviewer

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


### /Initialize Response Schema

Provided for reference if you're implementing Bootstrapping - the job of your server is to provide the values that Statsig's servers other wise would when they call /initialize. Statsig's getClientInitializeResponse function provides this payload.
Copy link

Choose a reason for hiding this comment

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

The word other wise appears to be a typo and should be written as otherwise in this sentence.

Spotted by Graphite Reviewer

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

@jasonwzm
Copy link
Member

@jasonwzm jasonwzm temporarily deployed to pull request December 27, 2024 00:05 Inactive
@brock-statsig brock-statsig merged commit 8e2409c into main Dec 27, 2024
3 checks passed
@brock-statsig brock-statsig deleted the combine-init-docs branch December 27, 2024 00:05
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