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

Does not work even after permissions reset #28

Closed
electriquo opened this issue Jun 12, 2023 · 17 comments
Closed

Does not work even after permissions reset #28

electriquo opened this issue Jun 12, 2023 · 17 comments
Labels

Comments

@electriquo
Copy link

I've installed Notify as follows

brew tap nahive/spotify-notify
brew install --cask spotify-notify --no-quarantine

Yet, Notify in unable to interact with Spotify (to show notification, play/stop, next/previous, etc.). Following the note in the README

$ tccutil reset AppleEvents
I executed the following command to reset all the permissions for Notify

tccutil reset All $(osascript -e 'id of app "Notify"')

Yet it did not solve anything and Notify still unable to interact with Spotify.

$ sw_vers
ProductName:		macOS
ProductVersion:		13.4
BuildVersion:		22F66

$ defaults read ~/Applications/Notify.app/Contents/Info.plist CFBundleShortVersionString
0.4.0

$ defaults read ~/Applications/Spotify.app/Contents/Info.plist CFBundleShortVersionString
1.2.13.661.ga588f749
@nahive nahive added the bug label Jun 12, 2023
@nahive
Copy link
Owner

nahive commented Jun 12, 2023

I've been able to replicate the issue, I'll fix it asap.

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

@foolioo i've found the issue, during rewrite, somehow entitlements got lost and were not included in the project. i'll push the updated version asap with refined permissions flow.

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

@foolioo should be working correctly. i've rewrote the permissions flow. please check it out 😄

@electriquo
Copy link
Author

electriquo commented Jun 12, 2023

@nahive Now the automation permission prompt was displaying and stop/play, next/previous and open are working

image

Clicking "Notification Settings" does not work

image
image

And notification appears only when clicking "Show notification" even when "Notify on play/pause" is checked and not when the keyboard shortcut is used.

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

Oh damn, somehow I've missed the correct urls it seems. Thank you for such a thorough testing! Will work on it asap.

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

@foolioo i've drafted new release, i hope this time works as expected :)

@electriquo
Copy link
Author

electriquo commented Jun 12, 2023

@nahive after upgrading to version 0.4.3 the application starts but does not appear in the status bar. does it work for you?

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

@foolioo that's strange, i seem to have this problem now too.

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

can you try removing the app and installing it again?

@electriquo
Copy link
Author

@nahive tried it. same behavior. what happens on your end when you install the app from brew?

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

it's kind of weird, for the first few times it hangs, and then if I compile the one from Xcode, run the compiled app once, the brew installed one starts working. I haven't changed the starting code so I'm wondering, maybe it hangs on notifications permissions..

and can't replicate it anymore.

@foolioo do you have the same problem if you install previous version?

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

@foolioo do you have the same problem if you install previous version?

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

Do you mind trying this one?
Notify.app.zip

@electriquo
Copy link
Author

@nahive Does it work for you?

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

i can't reproduce the bug anymore, so yes, both versions seem to be working correctly.

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

@foolioo is the 0.4.2 working correctly? and if so, does the binary above maybe fix the issue?

@nahive
Copy link
Owner

nahive commented Jun 12, 2023

if the issue persist, please open new issue as it’s separate from original one. thank you!

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

No branches or pull requests

2 participants