search

ulyssa / iamb

A Matrix client for Vim addicts
https://iamb.chat
Apache License 2.0
677 stars 46 forks source link

Notification mute not working #360

Open TheCodedProf opened 2 months ago

TheCodedProf commented 2 months ago

I am still getting all desktop notifications from channels I set to mentions only, and from what I can see it seems that changing the notification level of a room does not actually save.

I am running version 0.0.10 on NixOS and do have

[settings.notifications]
enabled = true

set in my config. I've attached a video displaying the issue.

https://github.com/user-attachments/assets/d7a30505-278a-4bef-8e82-8ccfb28a41c8

neektwothousand commented 2 months ago

same issue here, version 0.0.10 on archlinux

Andrew-Collins commented 1 month ago

Hi @neektwothousand @TheCodedProf,

I cannot replicate on 0.0.10 on archlinux (6.11.3-arch1-1), I am able to change the notification mode and notifications are parsed as they should be.

I have tried running from source (tag: v0.0.10, rust: 1.84.0-nightly (15fbd2f60 2024-10-08)), as well as installing iamb-bin from the aur (which just downloads and installs iamb-x86_64-unknown-linux-musl.tgz from the Github releases page).

Could you try creating a fresh room, something like :create ++alias=notify_test and then seeing if you can change the notifications settings in it?

Additionally can you set other properties of the rooms you were having trouble with (alias, topic, history, etc)?

distefam commented 3 weeks ago

Same issue here, on Fedora Linux running 0.0.10 installed with cargo.

distefam commented 3 weeks ago

Actually, I just verified that the setting works (verified via Element). It's the :room notify show command that doesn't work. And, desktop notifications still come from rooms that have been set to "mentions" even though I haven't been mentioned.