Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add documentation for the Copilot Insights Cmdlets #774

Merged
merged 8 commits into from
Dec 13, 2024
Merged

Add documentation for the Copilot Insights Cmdlets #774

merged 8 commits into from
Dec 13, 2024

Conversation

Shweta2024
Copy link
Contributor

@Shweta2024 Shweta2024 commented Dec 10, 2024 •
edited by Ruchika-mittal01
Loading

@Shweta2024
add documentation for Copilot Insights Cmdlets
ec78d61 
Add public documentation for Copilot Insights Cmdlets
@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit ec78d61:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

samkabue
samkabue suggested changes Dec 11, 2024
View reviewed changes
Copy link
Contributor

@samkabue samkabue left a comment

Choose a reason for hiding this comment

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

🕐

samkabue
samkabue suggested changes Dec 11, 2024
View reviewed changes
Copy link
Contributor

@samkabue samkabue left a comment

Choose a reason for hiding this comment

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

🕐

@samkabue samkabue added the Back to submitter The pull request didn't pass the review criteria and has been returned to the writer for updates label Dec 11, 2024
@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit d68ffa8:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit 6b3b9f9:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit bc5b1ff:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

@samkabue
Copy link
Contributor

samkabue commented Dec 11, 2024 •
edited
Loading

As these are new files, they need to be added to the table of contents.

See this PR #Closed

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit c15bb3a:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

@Shweta2024
Copy link
Contributor Author

Shweta2024 commented Dec 12, 2024 •
edited
Loading

As these are new files, they need to be added to the table of contents.

See this PR

hi @samkabue, I have added both the cmdlets to the table of contents. Please re-review and let me know if I need to make any improvements in this.

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit c2ae728:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

@Shweta2024 Shweta2024 requested a review from samkabue December 12, 2024 03:44
samkabue
samkabue approved these changes Dec 12, 2024
View reviewed changes
Copy link
Contributor

@samkabue samkabue left a comment

Choose a reason for hiding this comment

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

:shipit:

@samkabue samkabue added Sign off The pull request is ready to be reviewed and merged by PubOps and removed Back to submitter The pull request didn't pass the review criteria and has been returned to the writer for updates labels Dec 12, 2024
@Ruchika-mittal01 Ruchika-mittal01 self-assigned this Dec 12, 2024
@Ruchika-mittal01 Ruchika-mittal01 added the In review PubOps is reviewing the pull request label Dec 12, 2024
@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit cc68a3c:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

@Ruchika-mittal01
Copy link
Contributor

Hi @Shweta2024,

Following is a blocking issue that needs to be resolved before we can merge:

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md
sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md

  • All files are missing a TOC entry. If they are not intended for the TOC, they should include the required metadata.

Reference

Criteria used for pull request reviews

Blocking

  • When a new article is added, the same PR must also contain at least one updated TOC with an entry for the new article. Exceptions are allowed when the file has ROBOTS: NOINDEX, NOFOLLOW in the metadata.

After you resolve the blocking issues, remove the Back to submitter label and add the Sign off label again.
If you need help resolving issues in this PR, post your question in the Ask an Admin Teams Channel.

Thanks,
Ruchika

@Ruchika-mittal01 Ruchika-mittal01 added Back to submitter The pull request didn't pass the review criteria and has been returned to the writer for updates and removed In review PubOps is reviewing the pull request Sign off The pull request is ready to be reviewed and merged by PubOps labels Dec 12, 2024
@Shweta2024
Copy link
Contributor Author

Hi @Shweta2024,

Following is a blocking issue that needs to be resolved before we can merge:

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md

  • All files are missing a TOC entry. If they are not intended for the TOC, they should include the required metadata.

Reference

Criteria used for pull request reviews

Blocking

  • When a new article is added, the same PR must also contain at least one updated TOC with an entry for the new article. Exceptions are allowed when the file has ROBOTS: NOINDEX, NOFOLLOW in the metadata.

After you resolve the blocking issues, remove the Back to submitter label and add the Sign off label again. If you need help resolving issues in this PR, post your question in the Ask an Admin Teams Channel.

Thanks, Ruchika

Hi @Ruchika-mittal01 ,
I have made the entry in TOC and have already added both the new cmdlet references to the file sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md . Do you mean we still need to add it to some other place? Can you please share a reference?

@samkabue samkabue added Sign off The pull request is ready to be reviewed and merged by PubOps and removed Back to submitter The pull request didn't pass the review criteria and has been returned to the writer for updates labels Dec 13, 2024
@Ruchika-mittal01 Ruchika-mittal01 added the In review PubOps is reviewing the pull request label Dec 13, 2024
@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit d50a7b4:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/sharepoint-online/Start-SPOCopilotAgentInsightsReport.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

For any questions, please:

@Ruchika-mittal01 Ruchika-mittal01 merged commit 732046c into MicrosoftDocs:main Dec 13, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samkabue samkabue samkabue approved these changes

Assignees

@trent-green trent-green

@Ruchika-mittal01 Ruchika-mittal01

Labels
In review PubOps is reviewing the pull request Microsoft submitter Sign off The pull request is ready to be reviewed and merged by PubOps
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Shweta2024 @samkabue @Ruchika-mittal01 @trent-green