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

[Feature Request] Ability to use both IDs and names to save/export profiles #77

Open
Stwerp opened this issue Feb 8, 2023 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@Stwerp
Copy link

Stwerp commented Feb 8, 2023

** Description**
This may be intended behavior BUT it does not match use-case of the gif/video in the readme/github landing page.

konsave -l produces a list such as:

Konsave profiles:
ID            NAME
1              ProfileA
2              ProfileB

Attempting konsave -a 2 produces

Konsave: Profile not found :(
Please check the log at <$HOME>/.cache/konsave_log.txt for more details

It's unclear what the ID is used for. The landing page video shows the user typing konsave --apply 2 to apply a new profile.

I just installed konsave from pip today and its version 2.2.0

System information

  • running Rocky Linux 8 (red hat clone)
  • How did you install Konsave? using pip
  • What version of Konsave are you using? version 2.2.0
  • Did you try updating? no, but 2.2.0 looks like latest release per github sidebar

Again, this could be intended behavior, and then the issue is that the video is incorrect. Or I've messed something up on my end :) It's a new laptop I'm trying to get up and going.

@Prayag2
Copy link
Owner

Prayag2 commented Feb 9, 2023

This may be intended behavior

Yes, this indeed is the intended behavior. IDs were used in older versions and are now obsolete. I'll update the gif in the README and remove IDs from the output in the next release.

@ben2talk
Copy link

ben2talk commented Nov 27, 2023

Would it not be better, as is the case with 'yay' and 'paru' wrappers, to be able to (without recalling all of our profiles) simply do 'konsave -a' or 'konsave -r' or even 'konsave -s' and be offered a list, with the option to enter the ID to apply?

Konsave -a
ID	NAME
1	Cream23.11.27
2	Dark23.11.27
==> ID to apply (eg: 1 2 3, 1-3 or ^4)
==> 

Then

konsave -s
Konsave profiles:
ID	NAME
1	Cream23.11.27
2	Dark23.11.27
==> ID to overwrite (eg: 1 2 3, 1-3 or ^4) or ENTER to type name
==> 

@Prayag2
Copy link
Owner

Prayag2 commented Dec 17, 2023

Would it not be better, as is the case with 'yay' and 'paru' wrappers, to be able to (without recalling all of our profiles) simply do 'konsave -a' or 'konsave -r' or even 'konsave -s' and be offered a list, with the option to enter the ID to apply?

Amazing idea! I'm definitely going to implement this for the next release!

@Prayag2 Prayag2 added the enhancement New feature or request label Dec 17, 2023
@Prayag2 Prayag2 added this to the v2.3.0 milestone Dec 17, 2023
@Prayag2 Prayag2 changed the title Applying profile based on ID reports Profile Not Found [FEATURE REQUEST] Ability to use both IDs and names to save/export profiles Dec 17, 2023
@Prayag2 Prayag2 changed the title [FEATURE REQUEST] Ability to use both IDs and names to save/export profiles [Feature Request] Ability to use both IDs and names to save/export profiles May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants