Skip to content

Conversation

@rahulchhabria
Copy link
Contributor

"Logs Beta" checkbox functionality was integrated into SDK getting started configurations.

  • The logs option was added to src/components/onboarding/index.tsx, defining the new feature and its UI description.
  • OnboardingOptionButtons components were introduced to platform-includes/getting-started-config/*.mdx for Java, Go, PHP, Ruby, and JavaScript, enabling the "Logs Beta" checkbox for these platforms.
  • For Android and Flutter, existing OnboardingOptionButtons in docs/platforms/android/index.mdx and docs/platforms/dart/guides/flutter/index.mdx were updated to include the logs option.
  • These modifications ensure that selecting the "Logs Beta" checkbox conditionally displays the relevant logs setup code, which was already present within ___PRODUCT_OPTION_START___ logs and ___PRODUCT_OPTION_END___ logs markers in the respective .mdx files.
  • A new file, platform-includes/logs/setup/go.mdx, was also created for Go logs setup documentation.

@vercel
Copy link

vercel bot commented Jun 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
develop-docs ⬜️ Ignored (Inspect) Visit Preview Jun 12, 2025 4:22am
sentry-docs ⬜️ Ignored (Inspect) Visit Preview Jun 12, 2025 4:22am

@codecov
Copy link

codecov bot commented Jun 12, 2025

Bundle Report

Changes will increase total bundle size by 603 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-server-cjs 10.91MB 375 bytes (0.0%) ⬆️
sentry-docs-client-array-push 9.77MB 228 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js -3 bytes 1.64MB -0.0%
../instrumentation.js -3 bytes 973.36kB -0.0%
9523.js -3 bytes 949.31kB -0.0%
../app/[[...path]]/page.js 234 bytes 587.49kB 0.04%
../app/[[...path]]/page.js.nft.json 50 bytes 399.84kB 0.01%
../app/platform-redirect/page.js.nft.json 50 bytes 399.75kB 0.01%
../app/sitemap.xml/route.js.nft.json 50 bytes 397.72kB 0.01%

App Routes Affected:

App Route Size Change Total Size Change (%)
/[[...path]] 234 bytes 3.14MB 0.01%
view changes for bundle: sentry-docs-client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
static/chunks/pages/_app-*.js -3 bytes 873.24kB -0.0%
static/chunks/7750-*.js -3 bytes 401.83kB -0.0%
static/chunks/app/[[...path]]/page-*.js 234 bytes 80.49kB 0.29%
server/middleware-*.js 5.55kB 6.55kB 555.3% ⚠️
server/middleware-*.js -5.55kB 1.0kB -84.74%
static/cqalYuNc9U81AwvRtXVg2/_buildManifest.js (New) 616 bytes 616 bytes 100.0% 🚀
static/cqalYuNc9U81AwvRtXVg2/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/_6RCRGd8XAC1VXKlHjc3k/_buildManifest.js (Deleted) -616 bytes 0 bytes -100.0% 🗑️
static/_6RCRGd8XAC1VXKlHjc3k/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️

@codeowner-assignment codeowner-assignment bot requested review from a team June 12, 2025 04:22
@rahulchhabria rahulchhabria enabled auto-merge (squash) June 12, 2025 04:55
Copy link
Contributor

@codyde codyde left a comment

Choose a reason for hiding this comment

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

Looks good! Yay for logs!

@rahulchhabria rahulchhabria merged commit 60f32dc into master Jun 12, 2025
13 checks passed
@rahulchhabria rahulchhabria deleted the cursor/add-logs-beta-checkbox-functionality-27a8 branch June 12, 2025 06:20
@github-actions github-actions bot locked and limited conversation to collaborators Jun 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants