pgeu / pgeu-system

Mirror of the PostgreSQL Europe Conference and Membership management system.
MIT License
20 stars 28 forks source link

Shorten timestamps in reports #80

Closed andreasscherbaum closed 2 years ago

andreasscherbaum commented 2 years ago

The attendee report does not need to print the exact timestamp, down to the millisecond and time zone, most of the time.

Shorten the timestamp to either days or day + time (hour/minute), maybe make this a checkbox.

mhagander commented 2 years ago

Not sure we need to make it configurable -- but definitely stripping out millisecond and timezone we should do everywhere. Perhaps doing that will make it "small enough" that we don't need to add complexity beyond that.

andreasscherbaum commented 2 years ago

For the configuration option I was thinking "include time", but always strip out seconds, milliseconds and TZ. There might be use cases where the actual time is relevant, so have that as an option might help. Having the milliseconds in the report does not help anywhere.

mhagander commented 2 years ago

Let's start with just truncating it down to the minute level, and see after some real-world usage if that's enough or if we really need to also be able to truncate it to date only.

mhagander commented 2 years ago

Fixed in 8b30c7ce