Not storing API KEy - Githubissues

GeekyEggo / GoveeController

Govee controller for the Elgato Stream Deck.

GNU General Public License v3.0

42 stars 8 forks source link

Not storing API KEy #3

Closed thesach closed 2 years ago

thesach commented 2 years ago

Description

I have the API key entered and it works but then when I go to push the button some time later the key is not showing the button.

Reproduction Steps

Steps to reproduce the behaviour:

Go to '...'
Click on '...'
Scroll down to '...'
See error.

Expected Behaviour

A clear and concise description of what you expected to happen.

Logs and Screenshots

Please attach logs found at %appdata%\Elgato\StreamDeck\Plugins\com.geekyeggo.goveecontroller.sdPlugin\logs
If applicable, add screenshots to help explain the problem.

GeekyEggo commented 2 years ago

Hey @thesach, could I ask you to download the latest beta version from here and try using the plug-in again. When you encounter the problem, upload the logs and I’ll take a look.

GeekyEggo commented 2 years ago

Closing this as I’ve not heard anything; if the problem persists feel free to comment.

MikewrightJS commented 2 years ago

I actually have the same issue with the latest version, hit the error multiple times, which log should i upload?

GeekyEggo commented 2 years ago

Hey @MikewrightJS; if you try with the latest beta version, and if the problem persists upload the logs at %appdata%\Elgato\StreamDeck\Plugins\com.geekyeggo.goveecontroller.sdPlugin\logs I’ll take a look.

MikewrightJS commented 2 years ago

GoveeController.log

GeekyEggo commented 2 years ago

Thanks @MikewrightJS, I've updated the latest beta to help identify what's going on;

Improve button feedback based on settings.
Improve button feedback based on success response.
Improve logging of global API key existence.
Update request interception for missing API keys to not reset global settings.

Could I ask you download the latest beta, and repeat the steps above? Thanks for your help with getting to the bottom of this.

MikewrightJS commented 2 years ago

GoveeController.log

Hello no problem. After downloading the latest beta i am getting the same issue of an empty object in data from the api.

GeekyEggo commented 2 years ago

Hmm, so I understand correctly, is the flow:

Input API key successfully
See list of devices and setup action
Wait some time, see this error
Repeat from step 1?

MikewrightJS commented 2 years ago

I dont make it past option 1 (removed api key for the screenshot)

GeekyEggo commented 2 years ago

Good news @MikewrightJS, I think I might have found the root cause of the problem! Check out the latest beta and let me know if it fixes it for you.

MikewrightJS commented 2 years ago

Hello thanks for this. Works fine for me, it is not displaying the devices for me but i believe its because my device isnt actually supported by the Api! Unfortunate! Thanks for the fix though.