Skip to content

ref(debugger-modal): Display specific sentry-cli copy for react-native#86192

Merged
priscilawebdev merged 3 commits into
masterfrom
priscila/ref/debugger-modal/display-specific-copy-sentry-cli-react-native
Mar 4, 2025
Merged

ref(debugger-modal): Display specific sentry-cli copy for react-native#86192
priscilawebdev merged 3 commits into
masterfrom
priscila/ref/debugger-modal/display-specific-copy-sentry-cli-react-native

Conversation

@priscilawebdev

Copy link
Copy Markdown
Member

No description provided.

@priscilawebdev priscilawebdev requested a review from a team as a code owner March 3, 2025 14:52
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 3, 2025
!sourceResolutionResults.sdkName?.startsWith(
'sentry.javascript.react-native'
)
isReactNativeSDK({sdkName: sourceResolutionResults.sdkName})

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Hosting Publicly shall not be shown for react-native and since this is minor, I am already fixing it here in this PR

@@ -673,15 +676,24 @@ function HasDebugIdChecklistItem({
</p>
{defined(sourceMapsDocLinks.sentryCli) && (

@krystofwoldrich krystofwoldrich Mar 4, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we also update the text above?

From

It seems you already uploaded artifacts with Debug IDs, however, this event doesn't contain any Debug IDs yet. Generally this means that you didn't deploy the same files you injected the Debug IDs into. For Sentry to be able to show your original source code, it is required that you deploy the exact same files that you uploaded to Sentry.

To

It seems you already uploaded artifacts with Debug IDs, however, this event doesn't contain any Debug IDs yet. Generally this means that your application doesn't include the same files you uploaded to Sentry. For Sentry to be able to show your original source code, it is required that you build the application with the exact same files that you uploaded to Sentry.

cc: @smeubank

@@ -673,15 +676,24 @@ function HasDebugIdChecklistItem({
</p>
{defined(sourceMapsDocLinks.sentryCli) && (

@krystofwoldrich krystofwoldrich Mar 4, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

From

If you are using a [bundlerPluginRepoLink:Sentry Plugin for your Bundler], the plugin needs to be active when building your production app. You cannot do two separate builds, for example, one for uploading to Sentry with the plugin being active and one for deploying without the plugin. The plugin needs to be active for every build.

To bundlerPluginRepoLink = https://docs.sentry.io/platforms/react-native/manual-setup/metro/

The [bundlerPluginRepoLink:Sentry Metro Plugin] needs to be active when building your production app. You cannot do two separate builds, for example, one for uploading to Sentry with the plugin being active and one for deploying without the plugin. The plugin needs to be active for every build.

@krystofwoldrich krystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. 🚀

Thank you for fixing the source maps debugger for RN.

@priscilawebdev priscilawebdev merged commit e6b4021 into master Mar 4, 2025
@priscilawebdev priscilawebdev deleted the priscila/ref/debugger-modal/display-specific-copy-sentry-cli-react-native branch March 4, 2025 13:36
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 20, 2025
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.

3 participants