pikers / piker

(e2e) foss trading for non-tinas
GNU Affero General Public License v3.0
103 stars 17 forks source link

Fix crash on notification daemon not found #460

Closed guilledk closed 1 year ago

guilledk commented 1 year ago

Fix case where user has no notification daemon installed and an order clears, we crash due to notify-send command not present, previously @algorandpa tried to address this issue but only caught the case where the notify-send command exits with non zero exit code, now I add a try-catch around the offending code and log a warning on error, avoiding crash.