- 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
11 days 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.