- Issue created by @klaasvw
- π§πͺBelgium klaasvw
klaasvw β changed the visibility of the branch 3482699-created-date-in to hidden.
- Merge request !9915Format created date in the node author detail summary using the current locale β (Open) created by klaasvw
- π§πͺBelgium klaasvw
The MR addresses the issue by applying
Date.toLocaleDateString
to the date string. - πΊπΈUnited States smustgrave
Appears to have pipeline errors.
Issue summary appears to be missing some sections, particularly clear steps for reproducing the issue. May require test coverage.
- Status changed to Needs work
about 1 month ago 2:04am 2 April 2025 - π¦πΊAustralia acbramley
Tests all passed with the format change which tells me we lack any test coverage for the output. Went looking and confirmed, we have CollapsedSummariesTest but that's testing view port specific things. SettingSummariesContentTypeTest is testing it on the Node type itself. I've added NodeDetailsSummariesTest which covers the author and promoted option summaries
- πΊπΈUnited States smustgrave
Think breaking up to it's own test file should be fine. Looking in node javascript test folder some already seem to be very specific. Change LGTM.
- π«π·France nod_ Lille
The problem here is using the browser local and not the configured user interface local to format the date.
Have one editor use a browser configured in french, another editor with a US browser and they will never agree on what the created date is. I would be inclined to won't fix this. The current format is not pretty but it is never ambiguous.
- π¦πΊAustralia acbramley
That's a very good point, given the lack of activity here it doesn't seem like this is an overly desired change
Happy to close this one out.