Skip to content

Conversation

@chargome
Copy link
Member

@chargome chargome self-assigned this Nov 12, 2025
@chargome chargome requested a review from a team as a code owner November 12, 2025 10:30
@linear
Copy link

linear bot commented Nov 12, 2025

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 12, 2025
Copy link
Member

@priscilawebdev priscilawebdev left a comment

Choose a reason for hiding this comment

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

left a few comments, but in general it looks good! 🙌

import {getInstallCodeBlock} from 'sentry/gettingStartedDocs/node/utils';
import {t, tct} from 'sentry/locale';

export const getNodeMetricsOnboarding = <
Copy link
Member

Choose a reason for hiding this comment

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

Thank you for creating this file and for moving the function out of utils <3

I would just name this metrics and inline this part export const metrics = getNodeMetricsOnboarding({ in the index file.

@@ -1,4 +1,4 @@
import {getNodeMetricsOnboarding} from 'sentry/gettingStartedDocs/node/utils';
import {getNodeMetricsOnboarding} from 'sentry/gettingStartedDocs/node/metrics';
Copy link
Member

Choose a reason for hiding this comment

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

no tests for metrics?

@chargome chargome merged commit 16fc810 into master Nov 12, 2025
48 checks passed
@chargome chargome deleted the cg-onboarding-metrics-cf branch November 12, 2025 13:24
@chargome
Copy link
Member Author

I have a follow up pr already, will update it in there!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants