-
Notifications
You must be signed in to change notification settings - Fork 410
Same share folder notification displayed multiple times #679
Comments
I'm not sure what you mean, can you explain it more clearly? Maybe it is #672? |
I get the same thing: android client says "device M17 Laptop wants to share folder phone shares (id number)", I pick the folder to share, save and restart syncthing and within a minute it asks again the same thing. Also I select a second computer (the one I'm currently using) to share in the android syncthing settings, but when the subsequent repeat question comes back that second computer share is not selected |
I am having approximately this same issue. Sharing between two devices - an Android phone currently running 14.39 and a raspberry pi/raspbian running 14.40. The phone repeatedly gives the notification device [ras pi] wants to share folder [folder]. This is the only folder I am sharing to the phone. Happy to provide further info that would assist in diagnosing, particularly to determine if this is a bug or user error. |
I think this happens due to syncthing-android/app/src/main/java/com/nutomic/syncthingandroid/service/EventProcessor.java Line 120 in 7e3c6c0
|
Commit Catfriend1/syncthing-android@36fa58a fixes the creation of duplicate notifications about the same thing. It adds the device name instead of the device id for better readability. Notification logic for device, folder accepts is moved to NotificationHandler class. |
This problem also applies to device connection consent notifications. |
@alihammadraza @cedardocster @WillMorhler |
Thanks for the follow up and fix. Unfortunately, I'm not able to test the debug build but I do appreciate your efforts. |
Its weird we have to solve this kind of event silencing in all wrappers (webgui, syncthing-android, syncthing-macosx, etcetera) and it is not in the core accessable from the REST API. @AudriusButkevicius and @calmh should I open a issue/feature request for syncthing daemon to control it? |
I think (not sure) there is already one, essentially having and endpoint that lists pending invites. This way we don't have to rely on the events not rolling off. |
Probably you are pointing to this one: syncthing/syncthing#4084 |
No description provided.
The text was updated successfully, but these errors were encountered: