Skip to content

Commit

Permalink
Update limitations section in readme
Browse files Browse the repository at this point in the history
  • Loading branch information
exquo committed Sep 20, 2021
1 parent d71d561 commit 9e573c2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@
### Limitations

- Not yet supported by [signal-cli](https://github.com/AsamK/signal-cli/issues):
- *Sending* read receipts for received messages ([#231](https://github.com/AsamK/signal-cli/issues/231), [#305](https://github.com/AsamK/signal-cli/issues/305))
- Quoting a message ([#213](https://github.com/AsamK/signal-cli/issues/213))
- Adding @-mentions in sent messages ([#584](https://github.com/AsamK/signal-cli/issues/584))
- Voice calls ([#80](https://github.com/AsamK/signal-cli/issues/80))

- *Sending* read receipts for received messages.

This comment has been minimized.

Copy link
@moppman

moppman May 26, 2022

It's happening!! 😄
AsamK/signal-cli@27dbc67

edit: looking at the signal-cli code, sending read receipts has been possible for a while now (a message is identified by its timestamp). Using the new --send-read-receipts config flag in scli's daemon startup command would be easy, but sending read receipts on every message received isn't really what read receipts are about lol.

This comment has been minimized.

Copy link
@exquo

exquo Jun 3, 2022

Author Collaborator

Yes, signal-cli allowed sending read receipts for a while now, but only by launching a new signal-cli process, not by sending a dbus message to a continiously running signal-cli daemon, which is what we need for scli. See #102 (comment)

Using the new --send-read-receipts config flag in scli's daemon startup command would be easy, but sending read receipts on every message received isn't really what read receipts are about lol.

Exactly

- "View once" and "expiring" messages.
- See also: open [issues](https://github.com/isamert/scli/issues).

Expand Down

0 comments on commit 9e573c2

Please sign in to comment.