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

"Launch at login" doesn't work #646

Closed
5 of 6 tasks
simonbcn opened this issue Aug 16, 2024 · 6 comments
Closed
5 of 6 tasks

"Launch at login" doesn't work #646

simonbcn opened this issue Aug 16, 2024 · 6 comments
Assignees
Labels
bug Something isn't working fixed in next release The issue will be closed once next release is available

Comments

@simonbcn
Copy link

simonbcn commented Aug 16, 2024

Please confirm the following:

  • Be sure to check out FAQ
  • I have carefully read the README
  • I have searched through the issues page but couldn't find a solution.(Including issue that has been closed)
  • Easydict has been upgraded to the latest version
  • I understand and agree to the above, and understand that the project maintainer has limited energy, issues that do not follow the rules may be ignored or closed directly

Problem description

When I activate "launch at login" it doesn't really add it to the login items or start it when I reboot the system.

Is the issue consistently reproducible?

Reproducible

Steps to Reproduce

Read problem description

Screenshot

Captura de pantalla 2024-08-16 a las 10 07 11

Runtime log

com.izual.Easydict 2024-08-16--07-54-56-460.log

Expected result

It should add easydict to the login items on macOS.

Proposed solution (optional)

No response

Device type and OS version

Apple M2, macOS 14.5

Are you willing to submit a PR to fix this issue?

  • I'm willing to submit a PR to fix this issue
@simonbcn simonbcn added the bug Something isn't working label Aug 16, 2024
@tisfeng
Copy link
Owner

tisfeng commented Aug 16, 2024

Hello, please check the system settings - Privacy and Security - Automation, and ensure that Easydict has System Events permissions #625 (comment) .

If it already exists, try turning it off, then re-enable it, and then reopen the "Launch at login" item in Easydict.

@simonbcn
Copy link
Author

But this should not be so. Why doesn't it ask for permissions from the application itself when you activate that option?

@Jerry23011
Copy link
Collaborator

Because Easydict uses Apple Script with launch at login

fileprivate func updateLoginItemWithLaunchAtStartup(_ launchAtStartup: Bool) {

But this should not be so. Why doesn't it ask for permissions from the application itself when you activate that option?

This is a good point. Perhaps we can consider using other ways to implement launch at login. @tisfeng

This package works well.

@tisfeng
Copy link
Owner

tisfeng commented Aug 17, 2024

Thanks for the suggestion, it does need improvement here.

Using AppleScript for launch at login is not a good choice, it was just a solution for #64 and #79 .

Since we only support macOS 13+, it's time to switch to SMAppService.

LaunchAtLogin-Modern is a good option, I'll take a look at it later.

@tisfeng tisfeng added the fixed in next release The issue will be closed once next release is available label Aug 20, 2024
@tisfeng
Copy link
Owner

tisfeng commented Aug 20, 2024

This issue will be fixed in next release.

@tisfeng
Copy link
Owner

tisfeng commented Sep 15, 2024

Hello, the 2.9.0 version has fixed this issue.

@tisfeng tisfeng closed this as completed Sep 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed in next release The issue will be closed once next release is available
Projects
None yet
Development

No branches or pull requests

3 participants