-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[DO NOT MERGE] [WIP] Updates to monitor applications #9376
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
base: master
Are you sure you want to change the base?
Conversation
1d41aac
to
0a06866
Compare
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.
In essence, looks good! Just some minor suggestions.
|
||
Use one of these services to execute synthetic authentication requests. | ||
If you are using [rules](/rules) or [custom database connection(s)](/connections/database/custom-db) or other extensibility points, the synthetic transaction(s) should be configured to go through their logic. |
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 we be capitalizing Rules, Custom Database Connections, and Extensibility?
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.
will defer to tech writers
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.
@erjohnson what do you think about this one?
07389fc
to
65dc471
Compare
65dc471
to
5b46df3
Compare
5b46df3
to
f84bec3
Compare
No description provided.