search

cinnyapp / cinny

Yet another matrix client
https://cinny.in
GNU Affero General Public License v3.0
1.98k stars 257 forks source link

Desktop notifications don't contain the message #1883

Open Salamandar opened 3 months ago

Salamandar commented 3 months ago

Describe the bug

Up until the latest release, desktop notifications contained the beginning of the message. So you'd know if it was useful to go to the cinny tab. See https://github.com/cinnyapp/cinny/blob/9ecb233763048c730d24ddacecc8c002d3c8fc89/src/client/state/Notifications.js#L272

Now the notification just contains "new notification from X" (see https://github.com/cinnyapp/cinny/blob/e68c56b334107862ab2eec5635c11c40c050f00b/src/app/pages/client/ClientNonUIFeatures.tsx#L157)

Now you need to go into the cinny app to see if this notification is relevant or if it just interrupted your previous activity.

Reproduction

No response

Expected behavior

A useful notification with the content of the message, as before.

Maybe a configurable behaviour if you want to hide the content of the message for privacy purposes.

Platform and versions

Cinny version: v4.0

Additional context

No response

williamkray commented 3 months ago

would love for this to be configurable, similar to most messaging apps on mobile where you can choose whether to show content or keep it private.

regalialong commented 3 months ago

I brought this up in the chat at some point before and it's supposedly intentional, so I'll 1+ this issue.

Lxtharia commented 3 weeks ago

I also would love to have the option to hide/show the messages content in the notification so I can choose my own security/usability balance :)