Skip to content

Conversation

@EricDahlvang
Copy link
Member

@EricDahlvang EricDahlvang commented Jun 13, 2022

Fixes: #6357

@EricDahlvang EricDahlvang added the Automation: Parity with js The PR needs to be ported to JS label Jun 13, 2022
@EricDahlvang EricDahlvang requested a review from a team as a code owner June 13, 2022 20:45
@BruceHaley
Copy link
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Schema.dll

@tracyboehrer tracyboehrer merged commit 9d665d6 into main Jul 11, 2022
@tracyboehrer tracyboehrer deleted the eric/teamsReadReceipt branch July 11, 2022 13:26
lilyydu added a commit to microsoft/teams-ai that referenced this pull request Nov 8, 2023
## Linked issues

closes: #697 

## Details

- Added handling of read receipt events in personal scope, and
associated tests.

**Delegate:**
Task ReadReceiptHandler(ITurnContext turnContext, TState turnState,
ReadReceiptInfo data, CancellationToken cancellationToken)

**API:**
OnTeamsReadReceipt(ReadReceiptHandler<TState> handler)

**Reference:**
[BotBuilder
Code](https://github.com/microsoft/botbuilder-dotnet/blob/e4afa430ea73dd725be39a30d16fe361e77d07c8/libraries/Microsoft.Bot.Builder/Teams/TeamsActivityHandler.cs#L807)
[BotBuilder
PR](microsoft/botbuilder-dotnet#6356)


## Attestation Checklist

- [x] My code follows the style guidelines of this project

- I have checked for/fixed spelling, linting, and other errors
- I have commented my code for clarity
- I have made corresponding changes to the documentation (we use
[TypeDoc](https://typedoc.org/) to document our code)
- My changes generate no new warnings
- I have added tests that validates my changes, and provides sufficient
test coverage. I have tested with:
  - Local testing
  - E2E testing in Teams
- New and existing unit tests pass locally with my changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Automation: Parity with js The PR needs to be ported to JS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for Teams Read Receipts

3 participants