Problem/Motivation

Since this is a base field, token support isn't free. We need to add it. Should be fairly easy to do.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork unified_date-3455536

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

heddn created an issue. See original summary.

scott_euser made their first commit to this issue’s fork.

scott_euser opened merge request !2

scott_euser’s picture

scott_euser commented

Yeah I can see that being useful, thanks for flagging. Copy pasted across from node:created, seems to work fine. Gave it a manual test myself via metatag module just to check and seems to work okay + provide the full list of date formats like node:created/node:updated do. I don't fully get the bubble meta data use that Core's node.tokens.inc uses, but kept it in to avoid issues.

Does this look okay to you @heddn?

scott_euser’s picture

scott_euser commented
Status: Active » Needs review

Added a bit more test coverage to ensure the unified_date field is actually getting picked up by the token as well. Manual testing also working fine still.

I think assuming all tests pass now, should work well

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Needs review » Reviewed & tested by the community

This looks perfect!

scott_euser’s picture

scott_euser commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.