Skip to content
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

Notification OS X : use Apple Script Notification #551

Closed
wants to merge 1 commit into from

Conversation

wasimafser
Copy link

Depreceation of NSUserNotificationCenter causes problems with Mojave+ i guess

in reference with #449

Depreceation of NSUserNotificationCenter causes problems with Mojave+ i guess
Copy link

@beauwilliams beauwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works perfectly for myself. Reduces dependency. Would like to see this merged to main.

@akshayaurora
Copy link
Member

Current implementation has no issues on Monterey using venv. Using Python.org -> python 3.9+ .

Would be open for having this as a alternative fallback if original fails.

@0xSEM
Copy link

0xSEM commented May 2, 2023

Current implementation has no issues on Monterey using venv. Using Python.org -> python 3.9+ .

Would be open for having this as a alternative fallback if original fails.

Consider this implementation, as the existing implementation seems to fail when using PyInstaller on python 3.9+ on Ventura

@yeti9990
Copy link

yeti9990 commented Feb 6, 2024

Current implementation fails for me on:

Mac Studio M1
Mac Sonoma 14.2.1
plyer                2.1.0
pyobjus              1.2.3

This closed PR implementation worked for me.

@masterfelu
Copy link

this works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants