antiprism / mpd_oled

MPD, Volumio, RuneAudio and Moode OLED status and spectrum display for Raspberry Pi (and similar)
Other
168 stars 45 forks source link

Add option to provide additional date formats #61

Closed smeisner closed 3 years ago

smeisner commented 3 years ago

I am proposing a change to the -d option to require a parameter. The value provided after -d is used to set date_format. 0 and 1 are the same, but the new value of 2 indicates a date format of dd-Mmm-YYY (e.g. 27-Feb-2021).

antiprism commented 3 years ago

Hi Steve

Thanks for your suggestion and for preparing your code into a pull. I would prefer not to change display options at this time, and only added in the US date format as I know a different format can be confusing, It might be a while before I review display options, but when I do I will support a wide range of date formats and have made a note to include your suggested format.

Thanks

Adrian.

smeisner commented 3 years ago

ack. do you want me to close my PR?

antiprism commented 3 years ago

I can close it.