https://github.com/ms7m/notify-py/blob/f748189a2e8b4b147fcbf1acb6912c944d687671/notifypy/os_notifiers/linux.py#L33

• Added: notification urgency (low, normal, critical) for notify-send
• Fixed: notify-send, now is being used as the first choice if installed
• Fixed: OSs like BSD now default under the linux-behavioural category
• Fixed: issue under windows with extra kwargs that resulted in error
• Removed: linux_fallback_libnotify, no more need for it
• Replaced: functions with the in-built shutil.which function

PS. Tried not to be messy with the code and respect your overall structural patterns. It has been tested and works fine both on linux and on windows (+the in-built shutil.which function works with BSD too)

I am using parrot os. I have use the library for my project and everything works great except for the sound. At first it used .mp3 and it threw an error so i changed to .wav file and now i am not having errors but still when the notification comes, there is no sound.

When I try to make an exe of my python program with pyinstaller if the notification has audio an powershell prompt pops up. The paremeters I used were: --onefile --noconsole

Hi, I noticed a bug of notify-py in combination with PyInstaller on Windows (10): If a program using notify-py is compiled to an .exe with the --windowed flag, the powershell windows pop's up for second when a notification is shown.

This commit should fix it.

To reproduce:

1. Deps:

   pip install pyinstaller notify-py
   

2. Create test.py:

   from notifypy import Notify
   notification = Notify()
   notification.send(block=False)
   

3. Compile to .exe:

   pyinstaller --windowed test.py
   

4. Run ./dist/test/test.exe

Result:

When I am sending notification I get bunch of logging output into the terminal. Since this is a library people might not be really interested in this logging by default. For example, I am using it in CLI application and there it is very distracting and not required.

Can you please by default turn it off? Maybe it could be turned on with a flag in options of the constructor? Or environmental variable? I logging will be used only in debugging scenario.

First of all thanks for putting this together!

I wanted to use it for https://github.com/AuHau/toggl-cli but ran into problem where when I want to show notification I get:

Traceback (most recent call last):
...
  File "/Users/adam/Projects/me/toggl-cli/toggl/cli/commands.py", line 396, in notify
    notification = Notify()
  File "/Users/adam/Projects/me/toggl-cli/.direnv/python-3.8.1/lib/python3.8/site-packages/notifypy/notify.py", line 30, in __init__
    self._notifier = self._notifier_detect()
  File "/Users/adam/Projects/me/toggl-cli/.direnv/python-3.8.1/lib/python3.8/site-packages/notifypy/os_notifiers/macos.py", line 15, in __init__
    call_find_notificator = self._find_bundled_notificator()
  File "/Users/adam/Projects/me/toggl-cli/.direnv/python-3.8.1/lib/python3.8/site-packages/notifypy/os_notifiers/macos.py", line 40, in _find_bundled_notificator
    raise BinaryNotFound("afplay")
notifypy.exceptions.BinaryNotFound: Unable to find required afplay. Please check if it's installed.

I inspected the package and I see that it is still released with the snoretoast.exe and that the MacOs binary is not included. It seems bit weird but maybe you released old version?

It looks like the switch from setuptools to poetry broke the CLI script install, as that was only specified in setup.py. This change adds the appropriate [tool.poetry.scripts] entry to pyproject.toml so that will happen again:

~/code/notify-py (wisnij/20230102-poetry-install-cli=) $ pip install . -v
Using pip 22.3.1 from /usr/lib/python3/dist-packages/pip (python 3.10)
[...]
Successfully built notify-py
Installing collected packages: notify-py
  changing mode of /home/wisnij/.local/bin/notifypy to 755
Successfully installed notify-py-0.3.38

It has been some time since loguru's new release. There is not any breaking changes. It is better to update the version restriction on https://github.com/ms7m/notify-py/blob/c6f30e0784eb18d3b4424ede706719a5fd7cf7b9/pyproject.toml#L16

From Python documentation:

exception BaseException The base class for all built-in exceptions. It is not meant to be directly inherited by user-defined classes (for that, use Exception).

Bumps py from 1.9.0 to 1.10.0.

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Bumps py from 1.9.0 to 1.10.0.

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Hi! Thank you for this great packages!

It will be cool to have WSL support.

One way is to have an ability to send notifications using external tools i.e. wsl-notify-send:

For community

⬇️ Please click the 👍 reaction instead of leaving a +1 or 👍 comment

I'm trying to package this python module for a Linux distro and there are some issues I've noticed with your published packages:

1. Source releases don't include a copy of the MIT license the MIT license requires you provide a copy of the LICENSE file when you distribute your module
2. Some releases published on PyPI aren't published on GitHub retrieving the source releases for packaging from GitHub brings lower maintenance overhead and better transparency than retrieving them from PyPI
3. Release file naming isn't very packager friendly the module is named notify-py, yet the published release file names are notify_py, inconsistencies like this increase the verbosity of the packaging definitions and maintenance overhead

Reference

https://github.com/dynobo/normcap/issues/87

Looks like if a user manually "turns off" the notification sent by notify-py. They are unable to "re-enable". Need to find a workaround to allow the user to have the option to re-enable in the Windows Settings..

Current workaround is to simply change the application_name to something else each time you send a notification or when first initializing the Notify object..