pareeohnos / ktrade

A simple UI for managing your trades
MIT License
7 stars 5 forks source link

Improved notifications #28

Open pareeohnos opened 3 years ago

pareeohnos commented 3 years ago

Notifications in the app are a little messy at the moment. If for instance you place a huge order that is filled in multiple parts, you will get one notification for each part in quick succession.

Instead, the app should attempt to replace existing notifications an be clever. For example, if you place a huge order you should get one to say something like "Order filling", then it would update to say "Order filling: 1000/3000", then eventually update to success saying it's filled.

Dolivent commented 3 years ago

Also, when we trim, the status of the trade switches to 'canceled'. Should this field refer to the stop loss or something else?

pareeohnos commented 3 years ago

hm yeah that might be the stop loss cancellation interfering. Clearly a log bug. I'll open a new bug for that