Skip to content

Conversation

@sunilmathew
Copy link
Contributor

The field was wrongly mentioned as Reason Type instead of Resource Type. The description needs to be completed.

I'm not well-versed with this feature but came across this while preparing for DP-300 exam. If it is indeed Reason Type, please change the text above the table instead (where it is mentioned as Resource Type).

The field was wrongly mentioned as Reason Type instead of Resource Type. The description needs to be completed.

I'm not well-versed with this feature but came across this while preparing for DP-300 exam. If it is indeed Reason Type, please change the text above the table instead (where it is mentioned as Resource Type).
@prmerger-automator
Copy link
Contributor

@sunilmathew : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 967a890:

⚠️ Validation status: warnings

File Status Preview URL Details
azure-sql/managed-instance/alerts-create.md ⚠️Warning Details

azure-sql/managed-instance/alerts-create.md

  • Line 220, Column 26: [Warning: disallowed-html-tag - See documentation] HTML tag 'to' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@ttorble
Copy link
Contributor

ttorble commented Apr 15, 2025

@WilliamDAssafMSFT

Can you review the proposed changes? Note that the validation warning will need to be resolved before the PR can be merged.

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Apr 15, 2025
@WilliamDAssafMSFT
Copy link
Contributor

#reassign:MashaMSFT

@MashaMSFT
Copy link
Contributor

Hi @sunilmathew , thank you for your submission! I followed the steps in the article, and when I go to create an alert for SQL MI from the resource health pane, I see Reason type, as is described in the article:

image

Since the article is accurate, we'll be closing this PR, but we really appreciate you helping improve the documentation!

Thanks again, and I hope you have a great rest of your day!
Masha from the SQL Docs team
#please-close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants