Skip to content

Add user feedback integration checkbox #13959

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

Merged
merged 5 commits into from
Jun 9, 2025

Conversation

rahulchhabria
Copy link
Contributor

A "User Feedback" checkbox was added to the onboarding component, allowing users to opt-in to user feedback integration during setup.

Changes were made to src/components/onboarding/index.tsx:

  • The optionDetails object was updated to include a new 'user-feedback' entry. This defines the checkbox's display name ("User Feedback") and a descriptive tooltip: "Collect user feedback from anywhere in your application with an embeddable widget that allows users to report bugs and provide insights."
  • The OPTION_IDS array was extended to include 'user-feedback', ensuring the new checkbox appears in the setup flow alongside existing options like tracing and session replay.

This modification successfully integrates the new UI element. However, it was confirmed that the checkbox's state will not automatically update related code snippets. For code snippets to dynamically change, documentation files must explicitly wrap feedbackIntegration() code within <OnboardingOption optionId="user-feedback"> components or use product option markers like // ___PRODUCT_OPTION_START___ user-feedback. This session established the checkbox's presence and interactivity, but further steps are needed for full code snippet integration.

Copy link

vercel bot commented Jun 9, 2025

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

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 9, 2025 6:21am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Jun 9, 2025 6:21am
develop-docs ⬜️ Ignored (Inspect) Visit Preview Jun 9, 2025 6:21am

Copy link

codecov bot commented Jun 9, 2025

Bundle Report

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

Detailed changes
Bundle name Size Change
sentry-docs-server-cjs 10.86MB 226 bytes (0.0%) ⬆️
sentry-docs-client-array-push 9.77MB 229 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

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 235 bytes 80.15kB 0.29%
server/middleware-*.js -5.55kB 1.0kB -84.74%
server/middleware-*.js 5.55kB 6.55kB 555.3% ⚠️
static/jUAJ8eDmK4ivhVQqpRBFf/_buildManifest.js (New) 616 bytes 616 bytes 100.0% 🚀
static/jUAJ8eDmK4ivhVQqpRBFf/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/ILDn8vQEiDqsLhFFTs3rH/_buildManifest.js (Deleted) -616 bytes 0 bytes -100.0% 🗑️
static/ILDn8vQEiDqsLhFFTs3rH/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️

Files in static/chunks/app/[[...path]]/page-*.js:

  • ./src/components/onboarding/index.tsx → Total Size: 12.38kB
view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js -3 bytes 1.63MB -0.0%
../instrumentation.js -3 bytes 957.45kB -0.0%
9523.js -3 bytes 933.5kB -0.0%
../app/[[...path]]/page.js 235 bytes 586.37kB 0.04%

Files in ../app/[[...path]]/page.js:

  • ./src/components/onboarding/index.tsx → Total Size: 11.55kB

  • ./src/components/onboarding/index.tsx → Total Size: 1.39kB

App Routes Affected:

App Route Size Change Total Size Change (%)
/[[...path]] 235 bytes 3.13MB 0.01%

@rahulchhabria
Copy link
Contributor Author

@codyde it worked!! took a few prompts to get there but it worked!

@codyde codyde self-requested a review June 9, 2025 16:05
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.

This looks good!

We might consider in the future adding one of the AI rule snippets to the top like I did for logging and the general one to help folks quickly scaffold... but I dig this.

@codyde codyde merged commit 509080c into master Jun 9, 2025
15 checks passed
@codyde codyde deleted the cursor/add-user-feedback-integration-checkbox-2b3a branch June 9, 2025 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants