Skip to content

Made changes in channel-and-group-conversations.md#14323

Open
v-nikitach wants to merge 2 commits into
mainfrom
nikita-teamssdk-sample-code-updation7
Open

Made changes in channel-and-group-conversations.md#14323
v-nikitach wants to merge 2 commits into
mainfrom
nikita-teamssdk-sample-code-updation7

Conversation

@v-nikitach
Copy link
Copy Markdown
Collaborator

No description provided.

@microsoft-production-github-app

This comment was marked as outdated.

@learn-build-service-prod

This comment was marked as outdated.

@learn-build-service-prod

This comment was marked as outdated.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refreshes the "Channel and group chat conversations with a bot" doc to use the new Teams SDK APIs (e.g., app.OnMessage, AddMention, install.add lifecycle route) instead of the older Bot Framework SDK patterns (OnMessageActivityAsync, Mention object, conversationUpdate/teamMemberAdded). It also updates the description, date, and "See also" links, and removes the "Create new conversation threads" subsection in favor of pointing to proactive messaging.

Changes:

  • Replace Bot Framework SDK code samples (C#, TypeScript, Python) for retrieving and adding mentions with Teams SDK equivalents, and add JSON tabs where applicable with unique numbered tab IDs.
  • Replace the installation/welcome guidance with a Teams SDK install.add example, and update related "See also" links to Teams SDK references.
  • Update metadata (description, ms.date) and remove/comment out the v4-to-v3 include and the "Create new conversation threads" section.

Comment thread msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md Outdated
Comment thread msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md Outdated
Comment thread msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md Outdated
@microsoft-production-github-app
Copy link
Copy Markdown

Acrolinx Scorecards

We currently enforce a minimum score of 80.

Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article Score Issues Scorecard Processed
msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md 93 9 link

More info about Acrolinx

Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 2c263cc:

✅ Validation status: passed

File Status Preview URL Details
msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md ✅Succeeded View

For more details, please refer to the build report.

Copy link
Copy Markdown

@Harikrishnan-MSFT Harikrishnan-MSFT left a comment

Choose a reason for hiding this comment

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

Approved !!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants