feat(ai-conversations): onboarding#109651
Conversation
…structors Moves Set<string> type argument from variable declaration to constructor call to satisfy @typescript-eslint/consistent-generic-constructors rule. Also applies prettier formatting.
|
Looks good over all - one small thing I saw (not 100% that this is a problem): the "Other" snippet does not contain |
- Replace Header and OptionsWrapper styled components with Flex primitive from @sentry/scraps/layout per frontend guidelines - Fix "View Conversations" button not dismissing onboarding by exposing refetch from useShowConversationOnboarding and using it as the onDismiss callback instead of reloadProjects()
…nboardingUtils Deduplicate useOnboardingProject, PulseSpacer, PulsingIndicator, HeaderText, SubTitle, and BulletList between agents and conversations onboarding pages.
…egration snippet The "Other" (manual) Python integration was missing send_default_pii=True, which would cause LLM inputs/outputs to be filtered out.
…d replace SetupContent with Container Move serverSideNodeIntegrations to agentIntegrations.tsx as SERVER_SIDE_NODE_INTEGRATIONS export. Replace SetupContent styled component with Container primitive.
|
Good catch! You're right — the Python "Other" (manual) integration snippet was missing — Claude Code |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Part of: TET-1829: AI Conversations onboarding & docs
