nefertiti icon indicating copy to clipboard operation
nefertiti copied to clipboard

Published 20 hours ago • verified publisher icon svanas

Feat/limit sell orders

Open syzygy800 opened this issue 2 years ago • 2 comments

This feature could help a user to not lock-up too much capital. Especially, when the market drops. It could be made "notification only", however, I wanted to reduce the amount of new cli arguments. The flag could made a bit complexer like "--max-order=10,notificationonly". Not sure if that is acceptable as it is quite non-conform regarding the remaining flags.

syzygy800 avatar Jan 09 '22 12:01 syzygy800

Still no idea what was wrong with the else which was mentioned. However, the reading of the flag was in a wrong location anyhow. It doesn't need to be fetched in every iteration over the market list, just once. Further, now the --top value is adjusted for a market with: limit-opensells < top

syzygy800 avatar Jan 16 '22 10:01 syzygy800

Branch needed to a few modifications after rebasing from 196 to 202. Will open a new PR

syzygy800 avatar Feb 19 '22 11:02 syzygy800