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

Disable api key prompt if already set in main config #98

Closed
soifou opened this issue May 27, 2019 · 6 comments
Closed

Disable api key prompt if already set in main config #98

soifou opened this issue May 27, 2019 · 6 comments

Comments

@soifou
Copy link

soifou commented May 27, 2019

I'm using only the main config file (.wakatime.cfg) to set my api_key and some other settings I need for. I don't need to set any overriden value in Wakatime.sublime-settings.

However each time I open a new Sublime Text instance, Wakatime prompt me to enter my api key even if it does not prevent it from working as it should. So each time, I hit escape key to skip the prompt but this is kind of annoying.

Could you implement an option to not prompt for an api_key or review the logic behind this ? If this is not intended, I could give you some additionnal informations about my setup.

@alanhamlett
Copy link
Member

Implemented in v8.6.0 just released.

@soifou
Copy link
Author

soifou commented May 28, 2019

@alanhamlett

Thanks for the fix but it does still prompt with v8.6.0 :/
My WAKATIME_HOME is set to ~/.config/wakatime
My wakatime config ~/.config/wakatime/.wakatime.cfg
No user defined Wakatimesublime-settings

@alanhamlett
Copy link
Member

Sorry, probably because of a bug when parsing the config file. Should be fixed now in 8.6.1, can u see if it stops prompting after updating?

@alanhamlett alanhamlett reopened this May 29, 2019
@soifou
Copy link
Author

soifou commented May 29, 2019

The behaviour has changed.

Now, first time it continues prompting and creates a sublime-settings file with my api_key. The next time it stops prompting.

I don't want or need to manage several config files for my api key.

@alanhamlett
Copy link
Member

After upgrading to v8.7.0 you can delete your user sublime-settings file and restart Sublime for it to use the common config file without re-creating the sublime-settings file.

@soifou
Copy link
Author

soifou commented May 29, 2019

Yes this is now perfect, thanks a lot @alanhamlett !

@soifou soifou closed this as completed May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants