Skip to content

Updates to resolve UUF issue and nit edits #712

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

Merged
merged 4 commits into from
Jul 28, 2025

Conversation

msewaweru
Copy link
Collaborator

This pull request focuses on improving the clarity, consistency, and usability of documentation for the Microsoft Graph PowerShell SDK. The changes include updates to titles, descriptions, and examples across multiple files, as well as enhancements to syntax, formatting, and terminology to improve readability and user comprehension.

Copy link
Contributor

Learn Build status updates of commit 0b436d9:

⚠️ Validation status: warnings

File Status Preview URL Details
microsoftgraph/docs-conceptual/find-mg-graph-command.md ⚠️Warning View (graph-powershell-1.0 || graph-powershell-beta) Details
microsoftgraph/docs-conceptual/get-started.md 💡Suggestion View (graph-powershell-1.0 || graph-powershell-beta) Details
microsoftgraph/docs-conceptual/find-mg-graph-permission.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)

microsoftgraph/docs-conceptual/find-mg-graph-command.md

  • Line 182, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'authentication.md'.

microsoftgraph/docs-conceptual/get-started.md

  • Line 20, Column 86: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /graph/api/overview?view=graph-rest-1.0
  • Line 20, Column 163: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /graph/api/overview?view=graph-rest-beta

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.

Copy link
Contributor

Learn Build status updates of commit 1ab53d8:

💡 Validation status: suggestions

File Status Preview URL Details
microsoftgraph/docs-conceptual/get-started.md 💡Suggestion View (graph-powershell-1.0 || graph-powershell-beta) Details
microsoftgraph/docs-conceptual/find-mg-graph-command.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)
microsoftgraph/docs-conceptual/find-mg-graph-permission.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)

microsoftgraph/docs-conceptual/get-started.md

  • Line 20, Column 143: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /graph/api/overview?view=graph-rest-beta

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.

Copy link
Contributor

Learn Build status updates of commit c866767:

💡 Validation status: suggestions

File Status Preview URL Details
microsoftgraph/docs-conceptual/get-started.md 💡Suggestion View (graph-powershell-1.0 || graph-powershell-beta) Details
microsoftgraph/docs-conceptual/find-mg-graph-command.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)
microsoftgraph/docs-conceptual/find-mg-graph-permission.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)

microsoftgraph/docs-conceptual/get-started.md

  • Line 20, Column 143: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /graph/api/overview?view=graph-rest-beta

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.

Copy link
Collaborator

@omondiatieno omondiatieno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Left a few minor sugestions


```powershell
Connect-MgGraph -Scopes "User.Read.All","Group.ReadWrite.All"
```

The command prompts you to go to a web page to sign in with your credentials. Once you've done that, the command indicates success with a `Welcome To Microsoft Graph!` message. You only need to sign in once per session.
The command prompts you to go to a web page to sign in with your credentials. Once you've done that, the command indicates success with a `Welcome To Microsoft Graph!` message. You only need to sign in once per PowerShell session (that is, until you close and reopen your PowerShell window or call `Disconnect-MgGraph`).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The command prompts you to go to a web page to sign in with your credentials. Once you've done that, the command indicates success with a `Welcome To Microsoft Graph!` message. You only need to sign in once per PowerShell session (that is, until you close and reopen your PowerShell window or call `Disconnect-MgGraph`).
The command prompts you need to go to a web page to sign in with your credentials. Once you've done that, the command indicates success with a `Welcome To Microsoft Graph!` message. You only need to sign in once per PowerShell session (that is, until you close and reopen your PowerShell window or call `Disconnect-MgGraph`).

Co-authored-by: Jackline Omondi <57798781+omondiatieno@users.noreply.github.com>
Copy link
Contributor

Learn Build status updates of commit 8ff8e26:

💡 Validation status: suggestions

File Status Preview URL Details
microsoftgraph/docs-conceptual/get-started.md 💡Suggestion View (graph-powershell-1.0 || graph-powershell-beta) Details
microsoftgraph/docs-conceptual/find-mg-graph-command.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)
microsoftgraph/docs-conceptual/find-mg-graph-permission.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)

microsoftgraph/docs-conceptual/get-started.md

  • Line 20, Column 143: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /graph/api/overview?view=graph-rest-beta

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.

@msewaweru msewaweru merged commit 61161f4 into MicrosoftDocs:main Jul 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants