Skip to content

Update defender-for-cloud-planning-and-operations-guide.md #92

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Sbiz85
Copy link

@Sbiz85 Sbiz85 commented Jun 23, 2025

  • Fixed typos and grammar according to Microsoft Writing Style Guide
  • Restructured Roles section: condensed into one table
  • Slightly edited alt text
  • Included comments with rationale behind decisions as well as further questions that I'd ask for clarification
  • Will attach corrected graphic via email

- Fixed typos and grammar according to Microsoft Writing Style Guide
- Restructured Roles section: condensed into one table
- Included comments with rationale behind decisions as well as further questions that I'd ask for clarification
-Will attach corrected graphic via email
Copy link
Contributor

Learn Build status updates of commit f514d66:

✅ Validation status: passed

File Status Preview URL Details
articles/defender-for-cloud/defender-for-cloud-planning-and-operations-guide.md ✅Succeeded

For more details, please refer to the build report.

@v-regandowner
Copy link
Contributor

v-regandowner commented Jun 23, 2025

@Sbiz85 - Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

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

@batamig
Copy link
Contributor

batamig commented Jun 23, 2025

Copying comments and image from email for convenience

I have made the following changes:

  1. Corrected typos and grammar according to the Microsoft Writing Style Guide
  2. Ensured consistent content hierarchy (headers)
  3. Restructured the security roles section by condensing the content into one table
  4. Slightly edited some of the alt text to make the images more accessible to screen readers
  5. Included comments using that illustrates further questions as well as rationale behind my decisions

Two notes:

  1. The changes I made to the section regarding Azure roles was quite extensive: I restructured the entire thing, created a new subheader, and inserted a table. Coming from a UX background, it could be worthwhile to AB test how these changes could impact KPIs.

  2. One of my recommendations included fixing the typos and formatting in the graphic below. My main preference, however, would be to sit with the project lead and discuss possibly re-working the graphic to be less content-heavy and redundant, since the content is also spelled out below it in the article.

image

@Sbiz85
Copy link
Author

Sbiz85 commented Jun 23, 2025

@microsoft-github-policy-service agree [company="{your company}"]

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@Sbiz85
Copy link
Author

Sbiz85 commented Jun 23, 2025

@microsoft-github-policy-service agree

Copy link
Contributor

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

@v-regandowner
Copy link
Contributor

@dcurwin - Can you review the proposed changes?

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.

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.

5 participants