Summit-Events-App icon indicating copy to clipboard operation
Summit-Events-App copied to clipboard

Multi-day events has timezone formatted oddly in breadcrumb

Open hlhcg opened this issue 3 years ago • 3 comments

Multi-day events has timezone formatted oddly in breadcrumb

Screen Shot 2021-10-20 at 3 54 24 PM

hlhcg avatar Oct 20 '21 20:10 hlhcg

Coming back to this as I believe this was captured during the October Sprint. show an example of a time when the year transfers (new years) image

There is some inconsistency on where the year is placed compared to the time and other date formats.

Per the examples, the year in the second portion should come before the time and after the day date.

rdblake21 avatar May 10 '22 03:05 rdblake21

Handled on SummitEventsShared.cls, line 469-500

rdblake21 avatar May 10 '22 03:05 rdblake21

@tcdahlberg Please review and assess effort prior to packaging.

Ideally it would be: Friday, December 30, 2022, 8:00 AM - Monday January, 02, 2023 3:30 PM CDT Wednesday, October 20, 6:00 AM - Friday, October 22, 2022 9:00 AM CDT

So the Time and year need to be flipped on the second date to be consistent with the first.

rdblake21 avatar May 10 '22 03:05 rdblake21

image @rdblake21, I assume the commas after the year were intentional. Your comment above didn't have a comma after the year before the time. I like the way it is in your code change and approve.

tcdahlberg avatar Jan 24 '23 23:01 tcdahlberg

Included in beta release 0.28.0.1

tcdahlberg avatar Feb 16 '23 00:02 tcdahlberg

Included in production release 0.28.0.2

tcdahlberg avatar Feb 17 '23 18:02 tcdahlberg