-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
chore(docs): merge mobile SDK API docs #13840
Conversation
Drafted new singular mobile sdk api doc
Hi @ally-sassman 👋 Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days. Gatsby Cloud will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 20 to 30 minutes). |
✅ docs-website-develop deploy preview ready
|
``` | ||
recordBreadcrumb:(NSString* __nonnull)name attributes:(NSDictionary* __nullable)attributes; | ||
NewRelic.recordBreadcrumb(string $Name, map $eventAttributes) | ||
``` |
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 this be formatted to obj-c or swift?
|
||
## Requirements | ||
|
||
Agent version 5.13.0 or higher. |
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.
Can you confirm this version number is correct?
|
||
## Requirements | ||
|
||
Agent version 5.13.0 or higher. |
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.
Can you confirm this version number is correct?
## Requirements | ||
|
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.
@ndesai-newrelic Is there a capacitor version requirement to use this call?
## Requirements | ||
|
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.
@ndesai-newrelic Is there a cordova version requirement to use this call?
## Requirements | ||
|
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.
@ndesai-newrelic Is there a flutter version requirement to use this call?
## Requirements | ||
|
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.
@ndesai-newrelic Is there a react version requirement to use this call?
## Requirements | ||
|
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.
@ndesai-newrelic Is there a xamarin version requirement to use this call?
## Requirements | ||
|
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.
@ndesai-newrelic Is there a .NET MAUI version requirement to use this call?
## Syntax | ||
|
||
```csharp | ||
RecordBreadcrumb(string name, Dictionary<string, object> attributes): bool; |
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.
RecordBreadcrumb(string name, Dictionary<string, object> attributes): bool; | |
recordBreadcrumb(string name, Dictionary<string, object> attributes): bool; |
## Syntax | ||
|
||
```csharp | ||
RecordBreadcrumb(string name, Dictionary<string, object> attributes): bool; |
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.
RecordBreadcrumb(string name, Dictionary<string, object> attributes): bool; | |
recordBreadcrumb(string name, Dictionary<string, object> attributes): bool; |
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/mobile-sdk-api-guide.mdx
Outdated
Show resolved
Hide resolved
redirects: | ||
--- | ||
|
||
## Manual cross application tracing [#manual-cat] |
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.
This h2 looks a little clunky right next to the page title. Could it be preceded by some of the intro text?/ Is it necessary enough to include that it should replace the main title?
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.
Also, if this process is Android-specific, could that be made a bit clearer?
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/configure-settings.mdx
Show resolved
Hide resolved
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.
Reiterating Shawn's comments, this is super impressive work!! I mostly made suggestions for clarity &small fixes.
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/configure-settings.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/cross-app-tracing.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/cross-app-tracing.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/mobile-sdk-api-guide.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/name-interaction.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/name-interaction.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/network-request-failures.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/network-request-success.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/record-handled-exceptions.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/remove-all-attributes.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/remove-attribute.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/remove-attribute.mdx
Outdated
Show resolved
Hide resolved
|
||
### Objective-c [#objc] | ||
|
||
``` |
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.
``` | |
```objectivec |
|
||
### Swift [#swift] | ||
|
||
``` |
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.
``` | |
```swift |
|
||
### Objective-c [#objc] | ||
|
||
``` |
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.
``` | |
```objectivec |
|
||
### Swift [#swift] | ||
|
||
``` |
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.
``` | |
```swift |
## Return values [#return-values] | ||
|
||
Returns `true` if it succeeds, or `false` if it doesn't. | ||
|
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.
## Return values [#return-values] | |
Returns `true` if it succeeds, or `false` if it doesn't. |
Co-authored-by: ndesai-newrelic <[email protected]> Co-authored-by: Natasha White <[email protected]>
Changes include: