Skip to content
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

feat(feedback onboarding): add angular, astro, ember docs #65662

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

michellewzhang
Copy link
Member

@michellewzhang michellewzhang commented Feb 22, 2024

Adds Angular, Astro, and Ember onboarding docs for user feedback
Relates to #62792

@michellewzhang michellewzhang requested a review from a team February 22, 2024 21:28
@michellewzhang michellewzhang requested a review from a team as a code owner February 22, 2024 21:28
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 22, 2024
{
type: StepType.INSTALL,
description: tct(
'For the User Feedback integration to work, you must have the Sentry browser SDK package, or an equivalent framework SDK (e.g. [code:@sentry/react]) installed, minimum version 7.85.0.',
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
'For the User Feedback integration to work, you must have the Sentry browser SDK package, or an equivalent framework SDK (e.g. [code:@sentry/react]) installed, minimum version 7.85.0.',
'For the User Feedback integration to work, you must have the Sentry browser SDK package, or an equivalent framework SDK (e.g. [code:@sentry/angular]) installed, minimum version 7.85.0.',

Copy link
Member

Choose a reason for hiding this comment

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

could also be [codeAngular:@sentry/angular] or [codeIvy:@sentry/angular-ivy] as the replay snippet above

Copy link
Member

Choose a reason for hiding this comment

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

or we could rephrase this sentence... i don't have a strong suggestion for that though, just an option. I was looking at how it's phrased for the react onboarding and it's a bit different, which is fine to be the same or different too.

Copy link
Member Author

@michellewzhang michellewzhang Feb 22, 2024

Choose a reason for hiding this comment

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

i'm copying how it's phrased in the docs, but will incorporate the first suggestion for sure

@michellewzhang michellewzhang merged commit 4303b28 into master Feb 22, 2024
40 checks passed
@michellewzhang michellewzhang deleted the mz/onb-1 branch February 22, 2024 22:09
@github-actions github-actions bot locked and limited conversation to collaborators Mar 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants