-
Notifications
You must be signed in to change notification settings - Fork 232
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issues with notify.notify in release version #187
Comments
Now that I'm not getting Exceptions, it just fails silently, no notification produced. Dig down a bit, and directly called the generate applet (cachedir/alfred/package/Notificator.app..../applet) with various differing amounts of strings, regardless of what I input it silently fails (M1 Mac on 14.5 Sonoma). |
Okay reading around the issues more and I see this repo is py2.7 and the version on pypi is a fork for py3 support. Would be good to include disclaimer on repo/archive the repo if no more work will occur here! its still very high on google and linked to from everywhere! |
Two issues that I encountered with notify due to undocumented/wrong documented behavior.
Version downloaded via pip has different signature vs main/docs.
Pip version has signature
notify(title, subtitle, message, sound)
, and raises ValueError ifmessage == ''
, but main/docs have signaturenotify(title, subtitle, sound)
with value error iftitle == subtitle == ''
. I kept calling this function exactly as docs describe but getting ValueError...notify.notify requires you have a file at workflowdir/icon.png, otherwise you get somewhat cryptic error. It doesn't describe the need for an icon anywhere in this project's docs.
The text was updated successfully, but these errors were encountered: