Bryanzab/patterns alz docs update #23
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…g paths in Policy initiatives.
…e-alerts into bryanzab/patterns-alz-docs-update
…es.md to point to new .json paths for services.
JoeyBarnes
reviewed
Oct 5, 2023
| Only a small number of the resources support metric alert rules scoped at the subscription level and the metric alerts would only apply to resources deployed within the same region. The Support for Multiple Resources column to show which resources support metric alerts being scoped at the subscription level. For a complete list of which resources support metrics alert rules scoped at the subscription level click [here](https://learn.microsoft.com/en-us/azure/azure-monitor/alerts/alerts-types#monitor-multiple-resources). | ||
|
|
||
| > **NOTE**: There are hidden columns within the table, to scroll across you need to go to the bottom of the table to scroll and this is a limitation within tables in GitHub. If you have any suggestions to improving this expeirence please do get in touch via a PR or raise an issue, thank you. | ||
| > **NOTE**: We have tried to make it so that the table doesn't require a lot of side to side scrolling, but it is still a lot of information, we recommended that you click on the specifc alert name which will take you directly to alert details of that alert. |
There was a problem hiding this comment.
Clicking on the specific alert name takes them to the JSON for the policy template, not the alerts details page. Is that what you intended when you said, "will take you directly to the alert details of the alert."?
corrected indentation
changed multiline description to single line per YAML formatting
JoeyBarnes
requested changes
Oct 5, 2023
Updated Note based on Joey's comment.
bzabber
commented
Oct 5, 2023
There was a problem hiding this comment.
@JoeyBarnes Great feedback and I've updated the language in the note.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview/Summary
Updated alert-details.md:
Updated guidance and provided new information on how to download an extract of the alert details.
Policy-Initiatives.md
Breaking Changes
None
As part of this Pull Request I have
mainbranch