ethemguner / dobot

Real Time Crypto Currency Price Tracker by using Django
2 stars 0 forks source link

dont_buy and dont_sell stupidity #44

Closed ethemguner closed 2 years ago

ethemguner commented 2 years ago

while deciding sell or buy, system is looking for dont_buy and dont_sell flags.

understand the stupidity here?

we can delete them and put only one boolean field named "sell". if it's false then system will buy, if it's true then system will sell.

jesus. what i was thinking while writing this.

ethemguner commented 2 years ago

oh i remembered why i did this. i did this because o wanted to give a "neutral" state to deciding workflow. do not sell or buy, do nothing when both of them is False. but... still with adding a field named "active" boolean field we can manage that state. so...

RecNes commented 2 years ago

Do you mean adding on/off switch based on coin to stop/start whole operations at once?

ethemguner commented 2 years ago

yes, exactly. there should be a "do nothing" state. but not based on a coin, based on a DecisionSetting.