Tucsky / aggr

Cryptocurrency trades aggregator
https://charts.aggr.trade/
GNU General Public License v3.0
801 stars 232 forks source link

[BUG] Indicator Activity Dialog #378

Open adeacetis opened 8 months ago

adeacetis commented 8 months ago

This bug could lead to some enhancement of IndicatorLibraryDialog UX.

CreatedAt renders on a later date than UpdatedAt. I understand that the created field is related to the day, the indicator was saved into the browser database, instead, it should be set to a date prior. Could be arbitrarily set to two years ago as shown in IndicatorLibraryIndicator.

image

I am not a big proponent of having this information (createdAt and updatedAt) in a dialog. A 4th column could be used to store the updated date.

Also, we might have to add a createdAt to the defaultIndicatorList. So new indicators are actually dated properly and don't use the workspace installation date by default.

Also, it is noticed that the date and time are rendered using the En/Us format while the chart is using a 24-hour format.