-
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(Browser): browser doc audit #18935
Conversation
Hi @jeff-colucci 👋 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. We will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 10 to 20 minutes). |
✅ Deploy Preview for docs-website-netlify ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
|
@@ -69,30 +69,12 @@ These troubleshooting steps apply to problems when the browser monitoring agent | |||
|
|||
```html | |||
<script type="text/javascript"> | |||
(window.NREUM||(NREUM={})).loader_config={xpid:"VRUGVVJS";window.NREUM||(NREUM={}),__nr_require=function a(b,c,d){ ... | |||
;window.NREUM||(NREUM={});NREUM.init={ ... |
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.
@jeff-colucci Are the JavaScript examples incomplete on purpose?
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 seems to be the correct example based on the audit doc I got, the PR has been reviewed by the SMEs and it didn't come up but I will double check just to make sure before publishing
|
...ser/browser-monitoring/troubleshooting/troubleshoot-your-browser-monitoring-installation.mdx
Outdated
Show resolved
Hide resolved
...wser/troubleshooting/app-server-requests-greatly-outnumber-browser-pageview-transactions.mdx
Outdated
Show resolved
Hide resolved
...rowser/new-relic-browser/troubleshooting/browser-data-doesnt-match-other-analytics-tools.mdx
Outdated
Show resolved
Hide resolved
...rowser/new-relic-browser/troubleshooting/browser-data-doesnt-match-other-analytics-tools.mdx
Outdated
Show resolved
Hide resolved
.../docs/browser/new-relic-browser/troubleshooting/troubleshooting-session-trace-collection.mdx
Outdated
Show resolved
Hide resolved
…oubleshoot-your-browser-monitoring-installation.mdx Co-authored-by: ally sassman <42753584+ally-sassman@users.noreply.github.com>
…ubleshooting-session-trace-collection.mdx Co-authored-by: ally sassman <42753584+ally-sassman@users.noreply.github.com>
…wser-data-doesnt-match-other-analytics-tools.mdx Co-authored-by: ally sassman <42753584+ally-sassman@users.noreply.github.com>
…-server-requests-greatly-outnumber-browser-pageview-transactions.mdx Co-authored-by: ally sassman <42753584+ally-sassman@users.noreply.github.com>
|
…wser-data-doesnt-match-other-analytics-tools.mdx Co-authored-by: ally sassman <42753584+ally-sassman@users.noreply.github.com>
|
3 similar comments
|
|
|
Please follow conventional commit standards
in your commit messages and pull request title.
Give us some context
links to related docs, screenshots, etc.