Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

Description

Addresses review feedback from #55159 to add missing parameter descriptions to all CallInvite class constructors.

All five constructors now include proper XML documentation for their parameters, following the established pattern used throughout the Azure SDK codebase:

  • CallInvite(PhoneNumberIdentifier, PhoneNumberIdentifier) - Added descriptions for target phone number and caller ID parameters
  • CallInvite(CommunicationUserIdentifier) - Added description for Communication User target
  • CallInvite(MicrosoftTeamsUserIdentifier) - Added description for Microsoft Teams User target
  • CallInvite(TeamsExtensionUserIdentifier) - Added description for Teams Extension User target
  • CallInvite(MicrosoftTeamsAppIdentifier) - Added description for Microsoft Teams App target

This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

SDK Generation Guidelines

  • If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
  • The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
  • The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: v-dharmarajv <146493756+v-dharmarajv@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback on Microsoft Teams app identifier Add parameter descriptions to CallInvite constructors Jan 22, 2026
Copilot AI requested a review from v-dharmarajv January 22, 2026 19:37
@v-dharmarajv v-dharmarajv marked this pull request as ready for review January 22, 2026 19:43
@v-dharmarajv v-dharmarajv merged commit 510b053 into users/v-dharmarajv/microsoftTeamsAppSupport Jan 22, 2026
4 checks passed
@v-dharmarajv v-dharmarajv deleted the copilot/sub-pr-55159 branch January 22, 2026 19:43
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