Skip to content

Verify resolution of WithRecipient API review feedback#321

Closed
rido-min with Copilot wants to merge 1 commit into
rido/fix-fluent-TMfrom
copilot/sub-pr-318
Closed

Verify resolution of WithRecipient API review feedback#321
rido-min with Copilot wants to merge 1 commit into
rido/fix-fluent-TMfrom
copilot/sub-pr-318

Conversation

Copilot AI commented Feb 11, 2026

Copy link
Copy Markdown
Contributor

This PR documents verification that all review feedback from the targeted messaging API refactor has been successfully addressed in previous commits.

Verified resolutions:

  • Fluent API regression - MessageActivity.WithRecipient(Account) override added to preserve return type and method chaining (commit 77d0a9d)
  • Exception message accuracy - Validation error now shows correct Account parameter syntax instead of string (commit 77d0a9d)
  • Documentation consistency - README comparison table updated to use WithRecipient API throughout (commit 77d0a9d)
  • Test coverage - WithRecipient_MaintainsFluentChaining test added to prevent fluent API regressions (commit 77d0a9d)

No code changes were required. All feedback was already addressed in the commit history.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Simplify targeted messages with an overload in WithRecipient Verify resolution of WithRecipient API review feedback Feb 11, 2026
Copilot AI requested a review from rido-min February 11, 2026 15:25
@rido-min rido-min closed this Feb 11, 2026
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