-
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
Fix: Updated/Added details around APIs and include quickstarts #16855
Conversation
Added link to the Synthetics Quickstarts library on github pages Updated Broken Link and Certificate Check API types Added rows for Nerdgraph, Terraform under additional features, noted REST API deprecation
|
Hi @adammarshallNR 👋 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). If you add any more commits, you can comment |
✅ Deploy Preview for docs-website-netlify ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…, added style edits, and fixed table that was creating errors
Hi @adammarshallNR ! I've added @bpeckNR as a reviewer to ensure that we're all in line about updating this doc. I want to explain some of my changes:
Once Brian approves these changes, I might take the opportunity to give this doc a freshness update and soften out the edges. Thanks a bunch for filing this PR! |
id="EN5FkIW8YWk" | ||
type="youtube" | ||
/> | ||
You can view community-sourced examples of some synthetic monitors in our [synthetics quickstarts](https://newrelic.github.io/quickstarts-synthetics-library/#/). These examples are open source and not supported by New Relic, but may provide valuable examples of how to maximize your synthetic monitors view examples of |
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.
I like the call out about these examples being open source and not supported by New Relic. It looks like the sentence about the examples has an extra "view examples of" at the end. Maybe we should change it to something like this:
These examples are open source and not supported by New Relic, but contain valuable scripts and snippets that may help you cover common use cases in scripted API and scripted browser monitors.
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.
I like it! i'll add this in
Added link to the Synthetics Quickstarts library on github pages Updated Broken Link and Certificate Check API types Added rows for Nerdgraph, Terraform under additional features, noted REST API deprecation
Please follow conventional commit standards
in your commit messages and pull request title.
Give us some context
links to related docs, screenshots, etc.