Skip to content

feat: 🎸 Added read receipts#7

Merged
highesttt merged 2 commits into
mainfrom
1-feat-add-read-receipts-on-both-sides
Dec 20, 2025
Merged

feat: 🎸 Added read receipts#7
highesttt merged 2 commits into
mainfrom
1-feat-add-read-receipts-on-both-sides

Conversation

@highesttt

Copy link
Copy Markdown
Collaborator

No description provided.

@highesttt highesttt requested a review from Copilot December 20, 2025 06:56
@highesttt highesttt linked an issue Dec 20, 2025 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown

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 read receipt functionality to the LINE messenger bridge, enabling bidirectional read receipt synchronization between Matrix and LINE.

  • Added SendChatChecked method to send read receipts to LINE's API
  • Implemented handling for incoming LINE read receipts (operation type 55)
  • Added support for sending Matrix read receipts back to LINE via the ReadReceiptHandlingNetworkAPI interface

Reviewed changes

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

File Description
pkg/line/methods.go Adds SendChatChecked method to call LINE's sendChatChecked RPC endpoint for sending read receipts
pkg/connector/client.go Implements ReadReceiptHandlingNetworkAPI interface, adds operation type 55 handling for incoming read receipts, and implements HandleMatrixReadReceipt to forward Matrix read receipts to LINE

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

Comment thread pkg/line/methods.go
Comment thread pkg/connector/client.go Outdated

Copilot AI left a comment

Copy link
Copy Markdown

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 2 out of 2 changed files in this pull request and generated 2 comments.


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

@beeper beeper deleted a comment from Copilot AI Dec 20, 2025
@beeper beeper deleted a comment from Copilot AI Dec 20, 2025
@highesttt highesttt merged commit dd92122 into main Dec 20, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

feat: ✨ Add read receipts on both sides

2 participants