canonical / snapd-desktop-integration

User session helpers for snapd
GNU General Public License v3.0
17 stars 10 forks source link

installing missing theme snaps failed #52

Open fossfreedom opened 1 year ago

fossfreedom commented 1 year ago

On clicking our notification to install our theme snap on Lunar we receive an error notification saying the installation failed.

This is odd because looking at /snap we can see that our theme snap gtk-theme-pocillo has been downloaded and installed.

looking at journalctl I see this image

Ubuntu Budgie 23.04

Our default theme is Pocillo-dark.

fossfreedom commented 1 year ago

Further to this (v0.9 snapd-desktop-integration installed), clicking No to the notification actually falls through to try and install the theme snap.

https://github.com/snapcore/snapd-desktop-integration/blob/5960b4ae320d2d0a182b001d837016f7294db5f5/src/sdi-theme-monitor.c#L154 here a "default" is added to the yes/no answers and calling notify-cb -

so here https://github.com/snapcore/snapd-desktop-integration/blob/5960b4ae320d2d0a182b001d837016f7294db5f5/src/sdi-theme-monitor.c#L101 its trying to match either "yes" or "default" so is a "no" being translated as a "default" answer and thus the install routine being called?

kred-jon commented 3 months ago

fixed or not? because i received it the same problem