dscotese / kraken-grid

A bot that extends grid trading once you use it to create a grid.
GNU General Public License v3.0
9 stars 3 forks source link

Invisible testing mode = BAD! #45

Closed dscotese closed 2 months ago

dscotese commented 2 months ago

notest should be in the list of commands. The symptoms of being in any test mode should be more obvious.

dscotese commented 2 months ago

The fix for this is to update the code that handles input that is unrecognized. It currently asks to try it raw (meaning to send what was not recognized to the Kraken API server). If not that, then it simply sets process.TESTING to the first (space-delimited) word in the input. This turns on caching and test mode, which means we don't hit Kraken any more unless we don't already have an answer stored in the cache. Instead:

If the user declines to try the input raw, ask "Set process.TESTING to (whatever)?" If the user says yes, then ask "Use Caching (Y/N)" and do not accept a default.

dscotese commented 2 months ago

The fix for this (as described above) is in the fix39 branch.