Skip to content

feat: implement custom feedback form #435

Merged
lilyydu merged 5 commits intomainfrom
lilyydu/custom-feedback
Apr 22, 2026
Merged

feat: implement custom feedback form #435
lilyydu merged 5 commits intomainfrom
lilyydu/custom-feedback

Conversation

@lilyydu
Copy link
Copy Markdown
Contributor

@lilyydu lilyydu commented Apr 20, 2026

Custom Feedback Context:
Learn Doc

Copilot AI review requested due to automatic review settings April 20, 2026 22:16
@lilyydu lilyydu changed the title implement custom feedback form feat: implement custom feedback form Apr 20, 2026
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 adds support for Teams “custom feedback” loops by introducing a feedbackLoop channelData configuration (default vs custom), adding a new message/fetchTask invoke activity model + Apps handler registration, and updating the AI sample to demonstrate a custom feedback dialog flow.

Changes:

  • Add FeedbackLoop / FeedbackType models and normalize legacy feedbackLoopEnabled into feedbackLoop.
  • Add message/fetchTask invoke activity support (API model + Apps routing extension + deserialization support).
  • Update Samples.AI to use FeedbackType.Custom and return a task module dialog; add Apps tests for the new handler.

Reviewed changes

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

Show a summary per file
File Description
Tests/Microsoft.Teams.Apps.Tests/Activities/Invokes/Messages/FetchTaskActivityTests.cs Adds coverage to ensure OnMessageFetchTask routes and handlers are invoked correctly.
Samples/Samples.AI/Samples.AI.csproj Adds Cards project reference needed to build adaptive cards for the custom dialog.
Samples/Samples.AI/Program.cs Registers OnMessageFetchTask handler for custom feedback dialogs.
Samples/Samples.AI/Handlers/FeedbackHandler.cs Switches sample to FeedbackType.Custom and builds a task module adaptive card for feedback.
Libraries/Microsoft.Teams.Apps/Activities/Invokes/Messages/FetchTaskActivity.cs Adds attribute + OnMessageFetchTask route registration for message/fetchTask.
Libraries/Microsoft.Teams.Api/FeedbackLoop.cs Introduces FeedbackType string enum + FeedbackLoop configuration model.
Libraries/Microsoft.Teams.Api/ChannelData.cs Adds feedbackLoop property and documents legacy feedbackLoopEnabled.
Libraries/Microsoft.Teams.Api/Activities/Invokes/Messages/FetchTaskActivity.cs Adds the message/fetchTask activity model + reaction payload types.
Libraries/Microsoft.Teams.Api/Activities/Invokes/MessageActivity.cs Updates message invoke polymorphic (de)serialization to include fetchTask.
Libraries/Microsoft.Teams.Api/Activities/Activity.cs Normalizes feedback fields during WithData(ChannelData) and adds AddFeedback(FeedbackType) overload.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Libraries/Microsoft.Teams.Api/Activities/Activity.cs Outdated
Comment thread Libraries/Microsoft.Teams.Api/Activities/Activity.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 20, 2026 22:38
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

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Libraries/Microsoft.Teams.Api/Activities/Activity.cs
Comment thread Samples/Samples.AI/Handlers/FeedbackHandler.cs
@lilyydu lilyydu merged commit affd756 into main Apr 22, 2026
7 checks passed
@lilyydu lilyydu deleted the lilyydu/custom-feedback branch April 22, 2026 16:46
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.

3 participants