Skip to content

Conversation

chrisda
Copy link
Contributor

@chrisda chrisda commented Nov 22, 2024

And other formatting or schema fixes as needed

And other formatting or schema fixes as needed
Copy link

STOP Caution: Large pull request

This public repo applies limits on the number of files that can be merged in a single pull request (PR):

  • Warning Warning message and ability to unblock PR using the Merge large pull request* label - 30
  • Block Block message and PR is blocked - 100

This pull request will change 95 files, which exceeds the warning limit of 30.

Before this pull request can be merged, you must review the list of files it contains and their changes to confirm they are correct and intentional. This includes files you did not change and/or do not own. If there are changes you aren't familiar with, contact the individuals who submitted the commits included in this pull request and review the changes with them.

The individual who merges this pull request is responsible for ensuring the changes included in it are correct, and for resolving any issues that might result in merging it.

To merge this pull request

If you have determined that the changes in this pull request are correct, add the Merge large pull request* label to it, wait for the max/pr-file-count check to pass, and then click Merge. To add a label, click the gear icon next to Labels and then select Merge large pull request*.

Label UI screenshot

To reject this pull request

If you can't confirm the changes in this pull request are correct, click Close pull request at the bottom of this page to close it without merging. Work with your team to revert any changes that aren't correct.

* You must have write or triage access to this repo to add labels.

1 similar comment
Copy link

STOP Caution: Large pull request

This public repo applies limits on the number of files that can be merged in a single pull request (PR):

  • Warning Warning message and ability to unblock PR using the Merge large pull request* label - 30
  • Block Block message and PR is blocked - 100

This pull request will change 95 files, which exceeds the warning limit of 30.

Before this pull request can be merged, you must review the list of files it contains and their changes to confirm they are correct and intentional. This includes files you did not change and/or do not own. If there are changes you aren't familiar with, contact the individuals who submitted the commits included in this pull request and review the changes with them.

The individual who merges this pull request is responsible for ensuring the changes included in it are correct, and for resolving any issues that might result in merging it.

To merge this pull request

If you have determined that the changes in this pull request are correct, add the Merge large pull request* label to it, wait for the max/pr-file-count check to pass, and then click Merge. To add a label, click the gear icon next to Labels and then select Merge large pull request*.

Label UI screenshot

To reject this pull request

If you can't confirm the changes in this pull request are correct, click Close pull request at the bottom of this page to close it without merging. Work with your team to revert any changes that aren't correct.

* You must have write or triage access to this repo to add labels.

@chrisda chrisda added the Merge large pull request The pull request has more than 30 files, and the writer has confirmed that the changes are correct label Nov 22, 2024
Copy link

Learn Build status updates of commit 61864c6:

⚠️ Validation status: warnings

File Status Preview URL Details
teams/teams-ps/teams/New-CsAutoAttendantCallHandlingAssociation.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsCustomPolicyPackage.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsExternalAccessPolicy.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsHiddenMeetingTemplate.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsHiddenTemplate.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsMeetingTemplatePermissionPolicy.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTenantDialPlan.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsMeetingBrandingPolicy.md 💡Suggestion View (teams-ps) Details
teams/teams-ps/teams/Grant-CsUserPolicyPackage.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/Import-CsAutoAttendantHolidays.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/Import-CsOnlineAudioFile.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsApplicationAccessPolicy.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendant.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantCallableEntity.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantCallFlow.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantDialScope.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantMenu.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantMenuOption.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantPrompt.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchPolicyAssignmentOperation.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchPolicyPackageAssignmentOperation.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchTeamsDeployment.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsCallingLineIdentity.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsCloudCallDataConnection.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsEdgeAllowAllKnownDomains.md ✅Succeeded View (teams-ps)

This comment lists only the first 25 files in the pull request.

teams/teams-ps/teams/New-CsAutoAttendantCallHandlingAssociation.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'get-csonlineschedule'.

teams/teams-ps/teams/New-CsCustomPolicyPackage.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'get-csteamsmessagingpolicy'.
  • Line 0, Column 0: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/powershell/module/teams/https://learn.microsoft.com/microsoftteams/manage-policy-packages#what-is-a-policy-package' points to a learn site page that doesn't exist. Check the path or URL and update the link.

teams/teams-ps/teams/New-CsExternalAccessPolicy.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamsacsfederationconfiguration'.
  • Line 0, Column 0: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/powershell/module/teams//azure/communication-services/concepts/teams-interop' points to a learn site page that doesn't exist. Check the path or URL and update the link.

teams/teams-ps/teams/New-CsTeamsHiddenMeetingTemplate.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamsmeetingtemplatepermissionpolicy'.
  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamsmeetingtemplatepermissionpolicy'.

teams/teams-ps/teams/New-CsTeamsHiddenTemplate.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamstemplatepermissionpolicy'.
  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamstemplatepermissionpolicy'.

teams/teams-ps/teams/New-CsTeamsMeetingTemplatePermissionPolicy.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'get-csteamsfirstpartymeetingtemplateconfiguration'.

teams/teams-ps/teams/New-CsTenantDialPlan.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-cstenantdialplan'.

teams/teams-ps/teams/New-CsTeamsMeetingBrandingPolicy.md

  • Line 0, Column 0: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'szymonkatra' is not a valid Microsoft alias.

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

Learn Build status updates of commit fbb1fd1:

⚠️ Validation status: warnings

File Status Preview URL Details
teams/teams-ps/teams/New-CsAutoAttendantCallHandlingAssociation.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsCustomPolicyPackage.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsExternalAccessPolicy.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsHiddenMeetingTemplate.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsHiddenTemplate.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsMeetingTemplatePermissionPolicy.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTenantDialPlan.md ⚠️Warning View (teams-ps) Details
teams/teams-ps/teams/New-CsTeamsMeetingBrandingPolicy.md 💡Suggestion View (teams-ps) Details
teams/teams-ps/teams/Grant-CsUserPolicyPackage.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/Import-CsAutoAttendantHolidays.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/Import-CsOnlineAudioFile.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsApplicationAccessPolicy.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendant.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantCallableEntity.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantCallFlow.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantDialScope.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantMenu.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantMenuOption.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantPrompt.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchPolicyAssignmentOperation.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchPolicyPackageAssignmentOperation.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchTeamsDeployment.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsCallingLineIdentity.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsCloudCallDataConnection.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsEdgeAllowAllKnownDomains.md ✅Succeeded View (teams-ps)

This comment lists only the first 25 files in the pull request.

teams/teams-ps/teams/New-CsAutoAttendantCallHandlingAssociation.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'get-csonlineschedule'.

teams/teams-ps/teams/New-CsCustomPolicyPackage.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'get-csteamsmessagingpolicy'.
  • Line 0, Column 0: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/powershell/module/teams/https://learn.microsoft.com/microsoftteams/manage-policy-packages#what-is-a-policy-package' points to a learn site page that doesn't exist. Check the path or URL and update the link.

teams/teams-ps/teams/New-CsExternalAccessPolicy.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamsacsfederationconfiguration'.
  • Line 0, Column 0: [Suggestion: learn-site-link-broken - See documentation] Link 'https://learn.microsoft.com/powershell/module/teams//azure/communication-services/concepts/teams-interop' points to a learn site page that doesn't exist. Check the path or URL and update the link.

teams/teams-ps/teams/New-CsTeamsHiddenMeetingTemplate.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamsmeetingtemplatepermissionpolicy'.
  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamsmeetingtemplatepermissionpolicy'.

teams/teams-ps/teams/New-CsTeamsHiddenTemplate.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamstemplatepermissionpolicy'.
  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-csteamstemplatepermissionpolicy'.

teams/teams-ps/teams/New-CsTeamsMeetingTemplatePermissionPolicy.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'get-csteamsfirstpartymeetingtemplateconfiguration'.

teams/teams-ps/teams/New-CsTenantDialPlan.md

  • Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: 'set-cstenantdialplan'.

teams/teams-ps/teams/New-CsTeamsMeetingBrandingPolicy.md

  • Line 0, Column 0: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'szymonkatra' is not a valid Microsoft alias.

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

STOP Caution: Large pull request

This public repo applies limits on the number of files that can be merged in a single pull request (PR):

  • Warning Warning message and ability to unblock PR using the Merge large pull request* label - 30
  • Block Block message and PR is blocked - 100

This pull request will change 95 files, which exceeds the warning limit of 30.

Before this pull request can be merged, you must review the list of files it contains and their changes to confirm they are correct and intentional. This includes files you did not change and/or do not own. If there are changes you aren't familiar with, contact the individuals who submitted the commits included in this pull request and review the changes with them.

The individual who merges this pull request is responsible for ensuring the changes included in it are correct, and for resolving any issues that might result in merging it.

To merge this pull request

If you have determined that the changes in this pull request are correct, add the Merge large pull request* label to it, wait for the max/pr-file-count check to pass, and then click Merge. To add a label, click the gear icon next to Labels and then select Merge large pull request*.

Label UI screenshot

To reject this pull request

If you can't confirm the changes in this pull request are correct, click Close pull request at the bottom of this page to close it without merging. Work with your team to revert any changes that aren't correct.

* You must have write or triage access to this repo to add labels.

@github-actions github-actions bot removed the Merge large pull request The pull request has more than 30 files, and the writer has confirmed that the changes are correct label Nov 22, 2024
@chrisda chrisda added the Merge large pull request The pull request has more than 30 files, and the writer has confirmed that the changes are correct label Nov 22, 2024
Copy link

Learn Build status updates of commit 992b222:

💡 Validation status: suggestions

File Status Preview URL Details
teams/teams-ps/teams/New-CsTeamsMeetingBrandingPolicy.md 💡Suggestion View (teams-ps) Details
teams/teams-ps/teams/Grant-CsUserPolicyPackage.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/Import-CsAutoAttendantHolidays.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/Import-CsOnlineAudioFile.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsApplicationAccessPolicy.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendant.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantCallableEntity.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantCallFlow.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantCallHandlingAssociation.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantDialScope.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantMenu.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantMenuOption.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsAutoAttendantPrompt.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchPolicyAssignmentOperation.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchPolicyPackageAssignmentOperation.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsBatchTeamsDeployment.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsCallingLineIdentity.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsCloudCallDataConnection.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsCustomPolicyPackage.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsEdgeAllowAllKnownDomains.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsEdgeAllowList.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsEdgeDomainPattern.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsExternalAccessPolicy.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsGroupPolicyAssignment.md ✅Succeeded View (teams-ps)
teams/teams-ps/teams/New-CsHybridTelephoneNumber.md ✅Succeeded View (teams-ps)

This comment lists only the first 25 files in the pull request.

teams/teams-ps/teams/New-CsTeamsMeetingBrandingPolicy.md

  • Line 0, Column 0: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'szymonkatra' is not a valid Microsoft alias.

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:

@chrisda chrisda merged commit e42d030 into main Nov 22, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merge large pull request The pull request has more than 30 files, and the writer has confirmed that the changes are correct Microsoft submitter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants