Skip to content

Commit

Permalink
fix(cordova): fix docs around Cordova Perf and Profiling support (#8562)
Browse files Browse the repository at this point in the history
  • Loading branch information
kahest committed Nov 25, 2023
1 parent 1450e52 commit c63e60b
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,4 @@ Features:
- On-device symbolication for JavaScript (in `Debug` mode)
- [Ionic support](/platforms/javascript/guides/cordova/ionic/)
- Under the hood the SDK relies on our [JavaScript SDK](/platforms/javascript/), which makes all functions available for JavaScript also available in this SDK including:
- [Performance Monitoring](/product/performance/) to create transactions
- [Performance Monitoring](/platforms/javascript/guides/cordova/performance/instrumentation/custom-instrumentation) to create transactions
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
<SignInNote />

```javascript
onDeviceReady: function() {
var Sentry = cordova.require("sentry-cordova.Sentry");
Sentry.init({ dsn: '___PUBLIC_DSN___' });
}
```
3 changes: 1 addition & 2 deletions src/platforms/common/performance/index.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ supported:
- php
- node
notSupported:
- javascript.cordova
- native.breakpad
- native.crashpad
- native.minidumps
Expand Down Expand Up @@ -95,7 +94,7 @@ Learn more about performance monitoring <PlatformLink to="/configuration/options

## Verify

<PlatformSection supported={["react-native", "java.spring", "java.spring-boot", "android", "javascript", "apple", "dart", "rust"]}>
<PlatformSection supported={["react-native", "java.spring", "java.spring-boot", "android", "javascript", "apple", "dart", "rust"]} notSupported={["javascript.cordova"]}>

Verify that performance monitoring is working correctly by using our <PlatformLink to="/performance/instrumentation/automatic-instrumentation/">automatic instrumentation</PlatformLink> or by starting and finishing a transaction using <PlatformLink to="/performance/instrumentation/custom-instrumentation/">custom instrumentation</PlatformLink>.

Expand Down
1 change: 0 additions & 1 deletion src/platforms/common/performance/instrumentation/index.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ title: Instrumentation
sidebar_order: 20
description: "Learn how to instrument performance in your app."
notSupported:
- javascript.cordova
- native.breakpad
- native.crashpad
- native.minidumps
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ title: Automatic Instrumentation
sidebar_order: 10
supported:
- javascript
notSupported:
- javascript.cordova
description: "Learn what transactions are captured after tracing is enabled."
redirect_from:
- /performance/included-instrumentation
Expand Down
1 change: 1 addition & 0 deletions src/platforms/javascript/common/profiling/index.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ supported:
- javascript
notSupported:
- javascript.bun
- javascript.cordova
description: "Learn what transactions are captured after tracing is enabled."
---

Expand Down

1 comment on commit c63e60b

@vercel
Copy link

@vercel vercel bot commented on c63e60b Nov 25, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

sentry-docs – ./

sentry-docs-git-master.sentry.dev
sentry-docs.sentry.dev
docs.sentry.io

Please sign in to comment.