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

Remove Confusing String When Authenticating Through Password #55

Open
zakuArbor opened this issue Aug 30, 2024 · 1 comment
Open

Remove Confusing String When Authenticating Through Password #55

zakuArbor opened this issue Aug 30, 2024 · 1 comment

Comments

@zakuArbor
Copy link

zakuArbor commented Aug 30, 2024

There is a confusing string that is printed when you do not have the environment variable NEOCITIES_API_KEY set: Please login to get your API key:

$ neocities list
Please login to get your API key:
sitename: <random_site>
password: ••••••••••••••••••••••••••••••••••••

File: lib/neocities/cli.rb
Line: 72
Link:

puts "Please login to get your API key:"

Requested Solution: Remove this line

Reason: There will be users who will think they will need to place the API key instead of their password. I did not realize it was asking for a password because it mentioned API key before prompting me for my sitename and password. I do recognize in hindsight that it was asking for a password but it would ease potential friction for future users of the tool.

Helpful Issue: #31

p.s. thanks for the useful tool

@zakuArbor
Copy link
Author

zakuArbor commented Oct 7, 2024

Never mind, I now noticed the sentence asks you to login first before you can grab your API key. I misunderstood the prompt. Though it is not obvious as to why it is asking you to fetch your API key. Perhaps an additional explanation can be given to the string such as:

Please login to get your API key to enable autologin

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

No branches or pull requests

1 participant