proposed updates for flagsmith integration docs#14225
Closed
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
aliu39
commented
Jul 1, 2025
Comment on lines
+11
to
+15
| ### Set Up Evaluation Tracking | ||
|
|
||
| Flagsmith recommends using a OpenFeature SDK and [Flagsmith Provider](https://openfeature.dev/ecosystem?instant_search%5Bquery%5D=flagsmith&instant_search%5BrefinementList%5D%5Btype%5D%5B0%5D=Provider) to evaluate feature flags. To track these evaluations, visit one of our supported languages pages for OpenFeature: | ||
| * [JavaScript](/platforms/javascript/configuration/integrations/openfeature/) | ||
| * [Python](/platforms/python/integrations/openfeature/) |
Member
Author
There was a problem hiding this comment.
The only changed content for this file. The rest can stay the same, other than sidebar_order: 1
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Bug: Flagsmith Integration: Missing Save Step
The Flagsmith integration documentation contains two issues in the "Set Up Change Tracking" section:
- Step 1 incorrectly instructs users to "Select Generic in the dropdown that says 'Select a provider'". This should either be "Select Flagsmith" or include an explanation if "Generic" is the intended selection for Flagsmith integration.
- Step 3, "Set the Signing Secret", omits an explicit instruction to save the configuration, only stating "Once saved" without detailing the save action.
docs/organization/integrations/feature-flag/flagsmith/index.mdx#L26-L37
BugBot free trial expires on July 22, 2025
You have used $0.00 of your $50.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
6 tasks
Bundle ReportChanges will increase total bundle size by 860.64kB (4.08%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
Merge into #14126 (external PR)