Skip to content

Conversation

@Mohammed-MSFT
Copy link
Contributor

No description provided.

Copy link
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 introduces a comprehensive C# sample demonstrating how Microsoft Teams bots can handle shared channel events, specifically transitive member add/remove events in shared channels. The sample showcases bot functionality that responds to channel sharing/unsharing events and membership changes with visual feedback through adaptive cards.

Key Changes

  • Implements SharedChannelDataBot with event handlers for channel shared/unshared and member add/remove events
  • Creates adaptive card UI components for visualizing different event types with appropriate styling
  • Establishes comprehensive logging and debugging capabilities for shared channel event data

Reviewed Changes

Copilot reviewed 25 out of 46 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
samples/bot-shared-channel-events/csharp/SharedChannelEvents/Bots/SharedChannelDataBot.cs Core bot implementation with event handlers and adaptive card creation
samples/bot-shared-channel-events/csharp/SharedChannelEvents/Models/SharedChannelModel.cs Data models for shared channel events and membership information
samples/bot-shared-channel-events/csharp/SharedChannelEvents.csproj Project configuration with required NuGet packages
samples/bot-shared-channel-events/csharp/README.md Comprehensive documentation for setup and usage
README.md Updates to main repository documentation with new sample entry
.github/workflows/build-complete-samples.yml CI/CD configuration for building the new sample

"This sample shows how to build a Microsoft Teams bot using the Bot Framework SDK that responds to transitive member changes in shared channels. When a member is added to or removed from a parent team that shares a channel, Teams automatically updates the membership of the shared channel. The bot receives these transitive member add and remove events and can process them to track membership changes, maintain rosters, or trigger custom workflows."
],
"creationDateTime": "2025-10-09",
"updateDateTime": "2024-10-09",
Copy link

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

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

The updateDateTime (2024-10-09) is earlier than the creationDateTime (2025-10-09). The updateDateTime should be equal to or later than the creationDateTime.

Suggested change
"updateDateTime": "2024-10-09",
"updateDateTime": "2025-10-09",

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

}

.header-text {
font-size: 40x;
Copy link

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

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

Missing 'p' in font-size value. Should be '40px' instead of '40x'.

Suggested change
font-size: 40x;
font-size: 40px;

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

```bash
git clone https://github.com/OfficeDev/Microsoft-Teams-Samples.git

- Navigate to `samples/meetings-events/csharp`
Copy link

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

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

Incorrect path reference. Should be 'samples/bot-shared-channel-events/csharp' instead of 'samples/meetings-events/csharp'.

Suggested change
- Navigate to `samples/meetings-events/csharp`
- Navigate to `samples/bot-shared-channel-events/csharp`

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@Harikrishnan-MSFT
Copy link
Contributor

Copy link
Collaborator

@Pawank-MSFT Pawank-MSFT left a comment

Choose a reason for hiding this comment

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

looks correct, Please fix copilot comments before merging

@Mohammed-MSFT Mohammed-MSFT merged commit ab09243 into main Nov 4, 2025
1 check 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.

5 participants