-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: show error when use wrong API key #544
Conversation
@chrisleekr, did you test in dev mode? Launching the bot for local development via
|
@rando128 |
Hm, @rando128 are you sure you configured the API key? can you check again? |
Yeah, that's weird. My .env has all the keys setup:
my docker-composer.yml has:
I didn't change my setup, just pulled 0.094. And the APIError screen started to show up despite a functional environment. If it works on your side and nobody is facing the same issue I have, I will simply hack my |
Yeah, I would assume you would have the correct configuration. |
@rando128 @chrisleekr I double checked everything, generated new API keys but I still cant login to the bot interface and keep getting the new error screen - Failed to load your account information. Can you please help me fix it? Thanks! My env file is BINANCE_MODE=live BINANCE_TEST_API_KEY=fXQdqeK4aDCDx.... BINANCE_SLACK_ENABLED=false BINANCE_FEATURE_TOGGLE_NOTIFY_ORDER_CONFIRM=false BINANCE_AUTHENTICATION_ENABLED=true I have |
@rando128 yes, How to switch to master? |
Hum, AFAIK the bot doesn't use this library. So this is something else. Check this thread just in case. |
@rando128 Yeah Loom extension! |
@chrisleekr @rando128 |
I have same issues master work fine till i loud dompose.server.yml and all stop. |
Guys, thanks for the investigation. I will release a new version. |
Description
There have been several issues opened that the bot is stuck at loading. 99% of this issue is caused by using a wrong API key.
To provide more information, I've updated the front end to display an error message when the bot can't get account information.
Related Issue
#538
Motivation and Context
As description.
How Has This Been Tested?
Screenshots (if appropriate):