Skip to content

Update quickstart-onboard-machines.md #27

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

Closed
wants to merge 2 commits into from
Closed

Update quickstart-onboard-machines.md #27

wants to merge 2 commits into from

Conversation

burntsouup
Copy link

The process to connect your non-Azure machines to Microsoft Defender for Cloud from within the Azure Portal has changed. The UI within the Portal has also changed.

The process to connect your non-Azure machines to Microsoft Defender for Cloud from within the Azure Portal has changed.  The UI within the Portal has also changed.
Copy link
Contributor

Learn Build status updates of commit c396f82:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/defender-for-cloud/quickstart-onboard-machines.md ⚠️Warning Details

articles/defender-for-cloud/quickstart-onboard-machines.md

  • Line 51, Column 120: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/tutorial-enable-servers-plan#enable-on-an-azure-subscription-aws-account-or-gcp-project' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 51, Column 120: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/tutorial-enable-servers-plan#enable-on-an-azure-subscription-aws-account-or-gcp-project' will be broken in isolated environments. Replace with a relative link.

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:

@burntsouup
Copy link
Author

@microsoft-github-policy-service agree

Copy link
Contributor

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

replaced absolute link with relative link
Copy link
Contributor

Learn Build status updates of commit 0f3021c:

✅ Validation status: passed

File Status Preview URL Details
articles/defender-for-cloud/quickstart-onboard-machines.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@dcurwin
Copy link
Contributor

dcurwin commented Dec 22, 2024

Thanks for your contribution, but this PR was opened in the public repo, instead of the private one. I'm going to close it. If you'd like to make changes to the documentation, please use the Microsoft Learn Editor to ensure the PR opens in the public repo.

#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.

2 participants