-
Notifications
You must be signed in to change notification settings - Fork 906
Create New Tab Sample tab-channel-group-quickstart and tab-app-navigation (C#) #1713
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.
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
Conversation
There was a problem hiding this 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 C# implementations of two Microsoft Teams samples and updates image references in the TypeScript sample. The work involves creating complete C#/.NET versions of existing Node.js samples, including full application structures, configuration files, and Microsoft 365 Agents Toolkit integration.
- Adds complete C# implementations for Tab Channel Group QuickStart and Tab App Navigation samples
- Updates image references in TypeScript sample to use lowercase naming convention
- Provides comprehensive documentation and setup instructions for both new C# samples
Reviewed Changes
Copilot reviewed 60 out of 89 changed files in this pull request and generated 11 comments.
| File | Description |
|---|---|
| samples/tab-channel-group-quickstart/ts/* | Updates image references and documentation with lowercase filenames |
| samples/tab-channel-group-quickstart/csharp/* | Complete new C# implementation with web app, views, controllers, and M365 toolkit integration |
| samples/tab-app-navigation/csharp/* | Complete new C# implementation with navigation functionality and M365 toolkit support |
...ckstart/csharp/Microsoft.Teams.Samples.TabChannelGroupQuickstart.Web/Views/_ViewStart.cshtml
Outdated
Show resolved
Hide resolved
...b-app-navigation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/Views/_ViewStart.cshtml
Outdated
Show resolved
Hide resolved
...p-navigation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/Views/Shared/_Layout.cshtml
Outdated
Show resolved
Hide resolved
...ation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/Views/Home/TabAppNavigation.cshtml
Outdated
Show resolved
Hide resolved
...b-app-navigation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/Views/Home/Page3.cshtml
Outdated
Show resolved
Hide resolved
...b-app-navigation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/Views/Home/Page2.cshtml
Outdated
Show resolved
Hide resolved
...b-app-navigation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/Views/Home/Page1.cshtml
Outdated
Show resolved
Hide resolved
...b-app-navigation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/Views/Home/Index.cshtml
Outdated
Show resolved
Hide resolved
Pawank-MSFT
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please check PR feedbacks and also check copilot feedbacks
...ickstart/csharp/Microsoft.Teams.Samples.TabChannelGroupQuickstart.Web/Images/hello_world.png
Show resolved
Hide resolved
...rp/Microsoft.Teams.Samples.TabChannelGroupQuickstart.Web/Images/tab_channel_group_module.gif
Show resolved
Hide resolved
samples/tab-channel-group-quickstart/ts/Images/tab_channel_group_module.gif
Show resolved
Hide resolved
samples/tab-channel-group-quickstart/csharp/M365Agent/build/aad.manifest.local.json
Show resolved
Hide resolved
Pawank-MSFT
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks correct, Approving!
Pawank-MSFT
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix the small comments
samples/tab-app-navigation/csharp/Microsoft.Teams.Samples.TabAppNavigation.Web/appsettings.json
Outdated
Show resolved
Hide resolved
….com/OfficeDev/Microsoft-Teams-Samples into v-mfurquan/Irfan-sample-conversion-1
2.Tab App Navigation