search

wishhhmaster / CoinTNet

CoinT.Net - a C# trading tool for cryptocurrencies
42 stars 26 forks source link

Incorrect sort order #4

Closed thewarlok666 closed 10 years ago

thewarlok666 commented 10 years ago

Using a 2 digit date format (##/##/####) should fix this.

0 0 0 4 incorrect sort order

wishhhmaster commented 10 years ago

Cryptsy does not sort the orders by date automatically, unlike Bitstamp/BTC-e. I'll sort the orders in code. However, to display dates like ##/##/####, I need to now whether the user prefers to use dd/mm/YYYY or mm/dd/YYYY. Maybe I could just display them like YYYY-mm-dd? Otherwise, I could ask the user to select his date format in the options form.

thewarlok666 commented 10 years ago

You know the APIs better than I do. I do think making it select-able within the options screen would be preferable due to an individuals locale. Also, I think such a change should be displayed everywhere time/date info is displayed for consistency. This is kind of a minor issue but it does exist.

Peace

wishhhmaster commented 10 years ago

Yes, true. Also, on a related note, time for candles is UTC time. I'll probably need to add an option to display the data in the user's local time

thewarlok666 commented 10 years ago

This may help. No point in recreating the wheel...

https://www.google.com/#q=convert+utc+to+local+time+c%23

Peace

wishhhmaster commented 10 years ago

dcdd9da6e41277589af08997da7b21192cc409e3: I changed the display to YYYY-MM-dd HH:mm:ss, and the default sorting order is now by descending date

wishhhmaster commented 10 years ago

I'll add support for local times at a later stage