pwr22 / covbot

Matrix bot tracking SARS-COV-2 statistics.
https://matrix.to/#/!awMILboqesCENShaME:shortestpath.dev?via=shortestpath.dev&via=matrix.org&via=thomcat.rocks
MIT License
11 stars 1 forks source link

Send m.notice messages #21

Closed auscompgeek closed 4 years ago

auscompgeek commented 4 years ago

The m.notice type is primarily intended for responses from automated clients.

https://matrix.org/docs/spec/client_server/r0.6.0#m-notice

It is polite for bots to always send m.notice rather than m.text so that others can determine whether a message was from a bot.

pwr22 commented 4 years ago

It doesn't do this because the WhatsApp bridge doesn't propagate these

auscompgeek commented 4 years ago

Well, what if we added in a hack to send m.text if it's from a WhatsApp ghost?

pwr22 commented 4 years ago

That doesn't handle the case where I'm sending a message to a family member in a WhatsApp chat but from my matrix account 🤔.

Better would be to give the bridge m.notice support I think

On Wed, 25 Mar 2020, 08:17 David Vo, notifications@github.com wrote:

Well, what if we added in a hack to send m.text if it's from a WhatsApp ghost?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/pwr22/covbot/pull/21#issuecomment-603702221, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGEUMDG3HIWJVOWPIUWEGDRJG4ZVANCNFSM4LRQ2QJQ .

pwr22 commented 4 years ago

Added an issue to track doing this https://github.com/pwr22/covbot/issues/27.