-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Streamline kotlin onboarding docs #57342
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! 🚀
additionalInfo: ( | ||
<p> | ||
{tct( | ||
'To upload your source code to Sentry so it can be shown in stack traces, please refer to [link:Manually Uploading Source Context].', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we rewrite this to something like:
"If you prefer to manually upload your source code to Sentry, please refer to..." ?
This also applies to the Java wizard.
Or we could simply delete this part, WDYT?
'To upload your source code to Sentry so it can be shown in stack traces, please refer to [link:Manually Uploading Source Context].', | ||
{ | ||
link: ( | ||
<ExternalLink href="https://docs.sentry.io/platforms/java/source-context/" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe directly link to the manual steps: https://docs.sentry.io/platforms/java/source-context/#manually-uploading-source-context
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The introduction component (line 55) does not show up correctly. This is due to an open bracket that is not closed. Also, the strong component needs to be duplicated in order to work correctly. Here's my suggestion:
const introduction = (
<p>
{tct(
"Sentry supports Kotlin for both JVM and Android. This wizard guides you through set up in the JVM scenario. If you're interested in [strong:Android], head over to the [gettingStartedWithAndroidLink:Getting Started] for that SDK instead. At its core, Sentry for Java provides a raw client for sending events to Sentry. If you use [strong2:Spring Boot, Spring, Logback, JUL, or Log4j2], head over to our [gettingStartedWithJavaLink:Getting Started for Sentry Java].",
{
gettingStartedWithAndroidLink: (
<ExternalLink href="https://docs.sentry.io/platforms/android/" />
),
gettingStartedWithJavaLink: (
<ExternalLink href="https://docs.sentry.io/platforms/java/" />
),
strong: <strong />,
strong2: <strong />,
}
)}
</p>
);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch!
Last missing piece to close getsentry/sentry-java#2885
Move performance into "Next Steps", Use SAGP and Maven plugin, enable Source Context