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

[FEAT]: Add a command-line tool #466

Closed
DarylCantrell opened this issue Jan 3, 2023 · 1 comment
Closed

[FEAT]: Add a command-line tool #466

DarylCantrell opened this issue Jan 3, 2023 · 1 comment
Labels
feature New feature or request

Comments

@DarylCantrell
Copy link

Version

2.7.1

OS

Windows 11 / 64-bit / current

Device

Legion 5i Pro

Is your feature request related to a problem?

  • Currently the product doesn't have a usage story for non-admin users. For example, a family laptop with accounts for a parent (admin) and children (non-admin).

  • The process is run when a user logs in, which means none of the configurations are in place until login. Even if you start it minimized, the process runs in the system tray.

How would you like the problem to be solved?

Add a command-line tool. Ideally this would be placed in a non-user-specific folder like %ProgramFiles%.

The tool doesn't need any user interface; it could work purely based on a configuration file. That way an admin could set up a scheduled task which runs at machine startup. The task would turn on battery conservation, choose new keyboard colors, etc.

Since it's being run by the system these settings would apply for all users, or even when no user is logged in.

The usage pattern could be as simple as:

"%ProgramFiles%\Legion Toolkit\Apply-Config" my_global_config.json

What alternatives have you considered?

I don't think there's any way to apply configuration options globally using a UI. There's nowhere to put the UI.

Glancing at the code, it appears the UI and the Legion-specific library are fairly separate, so running this headless should not be too hard.

Additional information

No response

@DarylCantrell DarylCantrell added the feature New feature or request label Jan 3, 2023
@BartoszCichecki
Copy link
Owner

LLT is not able to work without admin rights that is true, but to function properly it also needs to run in background to change power profiles after Fn+Q or support Hybrid mode on gen 7 devices, or even handle 4-zone RGB keyboard.

Placing anything in Program Files will not fix any issues now. The only way out of this is to introduce a Windows service that will run as System and be available to all users. That is however a big rework of the architecture, so I am not sure when will I have time to do it.

There is already a similar issue reported: #246

@BartoszCichecki BartoszCichecki closed this as not planned Won't fix, can't repro, duplicate, stale Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants