- Issue created by @solideogloria
- @solideogloria opened merge request.
- Status changed to Needs review
almost 2 years ago 10:37pm 30 January 2023 - 🇺🇸United States asherry
@solideogloria I like the idea of being able to customize that date, but the "short" date is config that isn't actually required or locked. It's only default config that's installed on drupal install, and it can be deleted.
For a custom module that's fine, but for contrib it would be problematic, as any site with the short date format missing would then break when the code tries ->getPattern() on the NULL that's loaded from DateFormat::load('short')
- Status changed to Needs work
almost 2 years ago 11:10am 17 February 2023 - 🇺🇸United States asherry
I also think that the format "short" might not be the best assumption for an admin screen. I would imagine if some sites want to use a short format for node teasers, or taglines, they might not intuitively know that they're also changing what will show up for admins. On some larger scale sites these are even likely to be two different admins.
I would recommend maybe creating a default view that would be created on module install. (once this ticket is merged #3270975: Notification content is not available in Views → ) That view could be setup default with a new date format called "admin_notification_message", or something to that effect. That view can then replace the admin view that's currently in a controller.