Open
Conversation
|
Cursor Agent can help with this pull request. Just |
Co-authored-by: Jonathan Norris <jonathannorris@users.noreply.github.com>
d3577aa to
3d47a4d
Compare
Deploying devcycle-docs with
|
| Latest commit: |
3d47a4d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://69f26d79.devcycle-docs.pages.dev |
| Branch Preview URL: | https://cursor-devcycle-sdk-quality.devcycle-docs.pages.dev |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates DevCycle SDK documentation to correct event flushing option names/defaults, clarify OpenFeature context requirements (including anonymous-user behavior for web providers), and adjust Roku platform support/EdgeDB guidance.
Changes:
- Corrected event flush interval defaults (30s → 10s where applicable) and standardized flush interval option names in SDK docs.
- Clarified OpenFeature
targetingKey/targeting_key/user_idguidance across providers, including anonymous-user initialization behavior for Angular/React/JS OpenFeature providers. - Updated Roku docs for OS compatibility and EdgeDB enablement guidance; fixed minor JS docs link/typos.
Reviewed changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| docs/sdk/server-side-sdks/ruby/ruby-openfeature.md | Clarifies OpenFeature context requirements for the Ruby provider. |
| docs/sdk/server-side-sdks/ruby/ruby-gettingstarted.md | Corrects Ruby server SDK event flush default interval documentation. |
| docs/sdk/server-side-sdks/python/python-openfeature.md | Clarifies OpenFeature context requirements for the Python provider. |
| docs/sdk/server-side-sdks/php/php-openfeature.md | Clarifies OpenFeature context requirements for the PHP provider. |
| docs/sdk/server-side-sdks/node/node-openfeature.md | Clarifies OpenFeature context requirements for the Node provider. |
| docs/sdk/server-side-sdks/node/node-gettingstarted.md | Corrects Node server SDK event flush default interval documentation. |
| docs/sdk/server-side-sdks/nestjs/nestjs-openfeature.md | Clarifies OpenFeature context requirements for the NestJS provider. |
| docs/sdk/server-side-sdks/nestjs/nestjs-gettingstarted.md | Corrects NestJS server SDK event flush default interval documentation. |
| docs/sdk/server-side-sdks/java/java-openfeature.md | Fixes/standardizes OpenFeature targeting key terminology for Java provider docs. |
| docs/sdk/server-side-sdks/go/go-openfeature.md | Clarifies OpenFeature context requirements for the Go provider. |
| docs/sdk/server-side-sdks/dotnet/dotnet-openfeature.md | Clarifies OpenFeature context requirements for the .NET provider. |
| docs/sdk/client-side-sdks/roku/roku-usage.md | Fixes Roku flush interval option name and updates EdgeDB enablement guidance. |
| docs/sdk/client-side-sdks/roku/roku-install.md | Updates documented minimum supported Roku OS version. |
| docs/sdk/client-side-sdks/react/react-usage.md | Corrects React SDK flush interval option name in usage docs. |
| docs/sdk/client-side-sdks/react/react-openfeature.md | Updates OpenFeature targeting guidance; documents anonymous-user behavior. |
| docs/sdk/client-side-sdks/react-native/react-native-usage.md | Fixes invalid JS date usage and corrects flush interval option name; updates EdgeDB guidance. |
| docs/sdk/client-side-sdks/javascript/javascript-usage.md | Fixes internal link and corrects flush interval option name. |
| docs/sdk/client-side-sdks/javascript/javascript-openfeature.md | Updates OpenFeature targeting guidance; documents anonymous-user behavior. |
| docs/sdk/client-side-sdks/javascript/javascript-gettingstarted.md | Fixes initialization spelling/typos in deferred initialization section. |
| docs/sdk/client-side-sdks/ios/ios-usage.md | Corrects iOS flush interval option name in usage docs. |
| docs/sdk/client-side-sdks/ios/ios-gettingstarted.md | Updates iOS option name in initialization options table. |
| docs/sdk/client-side-sdks/flutter/flutter-usage.md | Corrects Flutter flush interval option name and updates EdgeDB guidance. |
| docs/sdk/client-side-sdks/angular/angular-gettingstarted.md | Updates Angular OpenFeature targeting guidance and anonymous-user wording. |
| docs/sdk/client-side-sdks/android/android-usage.md | Corrects Android flush interval option name and updates EdgeDB guidance. |
| docs/sdk/client-side-sdks/android/android-gettingstarted.md | Corrects Android proxy option names in initialization options table. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Corrected client SDK documentation regarding event flush intervals, OpenFeature
targetingKeyrequirements, and Roku OS compatibility.