-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Fix broken link in Cloudflare docs #14017
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Located the source of the Cloudflare
Here is the breadcrumbs boolean value that was correctly documented by cursor It appears cursor properly read this file, or had some kind of RAG knowledge of it. The |
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
Outdated
Show resolved
Hide resolved
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.
Left a couple suggestions, looks good!
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…tchIntegration.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
Bundle ReportChanges will increase total bundle size by 270 bytes (0.0%) ⬆️. 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:
|
[Cursor created PR - Reviewed by me]
Broken links in the Cloudflare configuration documentation have been resolved.
docs/platforms/javascript/common/configuration/integrations/fetchIntegration.mdx
, was created.fetchIntegration
in the integrations table.Sentry.fetchIntegration
, configuration options (breadcrumbs
,shouldCreateSpanForRequest
), and usage examples, specifically noting its relevance to Cloudflare Workers.requestDataIntegration
inplatform-includes/configuration/integrations/javascript.cloudflare.mdx
was updated../requestDataIntegration
, had incorrect casing and pointed to a non-existent path../requestdata
to match the actual filenamerequestdata.mdx
, ensuring consistency with other integration links.All links in the Cloudflare integrations table now correctly point to existing documentation files.