fix: use notification plugin

[Kruemmelspalter]

This makes desktop notifications work. For whatever reason, the cinny icon is shown when using new Notification("...") in the debug console, but not when triggered from the cinny app itself.

Also, requesting permission does nothing right now and the "Enable" button is always shown, this may be because the plugin makes window.Notification.requestPermission an async function and it's being called form sync code, but overriding it to be a sync function didn't help either.

[f-krause]

This feature would be greatly appreciated to be released soon!! :)

[kfiven]

This feature would be greatly appreciated to be released soon!! :)

@f-krause https://github.com/cinnyapp/cinny-desktop/actions/runs/24305479149 Test it out on test build. From my testing, it did not fix it.

[Kruemmelspalter]

for me it works from cargo tauri dev on wayland/niri, but it doesn't when using cargo tauri build --no-bundle and then running the output binary 🤔

[Serizao]

On my side this modification seem be work : Serizao@566a83e

[Kruemmelspalter]

ah, maybe Notification.requestPermission just doesn't do anything in dev mode but needs to be called in release mode

[Kruemmelspalter]

yes, with this it should work now

[Kruemmelspalter]

@kfiven idk why i can't request a review but this would be ready now i suppose

[Chr1s70ph]

I tested it with cargo tauri dev on MacOS and the issue was not fixed

[Kruemmelspalter]

I don't have access to a mac so sadly I can't really test / diagnose / fix that