-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
feat(flags): show ff index and generic integration doc for serverside JS #14328
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
base: master
Are you sure you want to change the base?
Conversation
…s, update generic page
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
||
<Alert level ="info"> | ||
|
||
If you are using an external feature flag provider, refer to the [supported list](/platforms/javascript/feature-flags/#enable-evaluation-tracking) before setting up this integration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This alert was removed because we already link back to the eval tracking section from "next steps"
Bundle ReportChanges will increase total bundle size by 22.72kB (0.1%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Missing Feature Flag Change Tracking Index Files
The includePath
for feature flag change tracking documentation was updated from feature-flags/change-tracking-list
to feature-flags/change-tracking-index
in JavaScript, Python, and product issues documentation. However, the new feature-flags/change-tracking-index
files are missing from this commit, resulting in missing content.
docs/platforms/javascript/common/feature-flags/index.mdx#L17-L18
sentry-docs/docs/platforms/javascript/common/feature-flags/index.mdx
Lines 17 to 18 in 376f787
<PlatformContent includePath="feature-flags/change-tracking-index" /> |
docs/platforms/python/feature-flags/index.mdx#L17-L18
sentry-docs/docs/platforms/python/feature-flags/index.mdx
Lines 17 to 18 in 376f787
<PlatformContent includePath="feature-flags/change-tracking-index" /> |
docs/product/issues/issue-details/feature-flags/index.mdx#L62-L63
sentry-docs/docs/product/issues/issue-details/feature-flags/index.mdx
Lines 62 to 63 in 376f787
<PlatformContent includePath="feature-flags/change-tracking-index" platform="_default"/> |
Was this report helpful? Give feedback by reacting with 👍 or 👎
Re: cursor - |
In 9.31.0 JS SDK we started supporting

featureFlagsIntegration
in node. This PR shows the FF index page in node platforms with only this integration linked:And shows the
featureFlagsIntegration
page for node platforms, with 9.31.0 as the min version.Deno (server):

React (browser):
