Skip to content

Freshness navigating graphps #661

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 6 commits into from
Mar 17, 2025

Conversation

msewaweru
Copy link
Collaborator

This pull request includes changes to improve the clarity and accuracy of the documentation for navigating the Microsoft Graph PowerShell SDK. The most important changes include updating the title and customer intent, correcting API references, and refining the language for better readability.

Documentation improvements:

Copy link
Contributor

Learn Build status updates of commit 9cc2dfb:

⚠️ Validation status: warnings

File Status Preview URL Details
microsoftgraph/docs-conceptual/navigating.md ⚠️Warning View (graph-powershell-1.0 || graph-powershell-beta) Details

microsoftgraph/docs-conceptual/navigating.md

  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 13, Column 1: [Warning: h1-not-first - See documentation] Markdown content is not allowed before H1 'Navigate the Microsoft Graph PowerShell SDK'.
  • Line 1, Column 1: [Suggestion: ms-date-missing - See documentation] Missing required attribute: 'ms.date'.
  • Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

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:

Copy link
Contributor

Learn Build status updates of commit 58e81eb:

✅ Validation status: passed

File Status Preview URL Details
microsoftgraph/docs-conceptual/navigating.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)

For more details, please refer to the build report.

For any questions, please:

@SteveMutungi254
Copy link
Contributor

@msewaweru : I have reviewed and left suggestions via this PR - msewaweru#1

You can merge the changes if happy with the suggestions.

Review suggestions - Freshness navigating graphps
@msewaweru
Copy link
Collaborator Author

@SteveMutungi254 - Thanks for the updates I have merged them into this PR. Please rereview.

Copy link
Contributor

@SteveMutungi254 SteveMutungi254 left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Contributor

Learn Build status updates of commit c7ca06b:

✅ Validation status: passed

File Status Preview URL Details
microsoftgraph/docs-conceptual/navigating.md ✅Succeeded View (graph-powershell-1.0 || graph-powershell-beta)

For more details, please refer to the build report.

For any questions, please:

@msewaweru msewaweru merged commit ddc1a97 into MicrosoftDocs:main Mar 17, 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