Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Date format has changed for fields in newer versions of dotCMS #25008

Closed
swicken-dotcms opened this issue May 22, 2023 · 4 comments · Fixed by #25017
Closed

Date format has changed for fields in newer versions of dotCMS #25008

swicken-dotcms opened this issue May 22, 2023 · 4 comments · Fixed by #25017
Labels
LTS: Excluded Ticket that has been excluded from at least one LTS OKR : Customer Success Owned by Arno OKR : Customer Support Owned by Scott QA : Approved QA : Passed Internal Release : 23.01.4 Included in LTS patch release 23.01.4 Release : 23.06 Team : Scout Type : Defect

Comments

@swicken-dotcms
Copy link
Contributor

Parent Issue

No response

Problem Statement

The date format being returned has changed when requesting content from the API. This only applies to fields, as system dates are still returning in the old format

Here's an example from a demo starter blog post:

"publishDate": "2022-12-27 21:52:10.419",
"postingDate": "Fri Sep 18 21:33:00 UTC 2020",

Publish date is a system field, which is showing the old format.
Posting date is a content type field, which is showing the new (presumably incorrect) format.

This will cause all sorts of issues for anyone upgrading to a version impacted that utilizes calls to our API.

Steps to Reproduce

  1. Start up dotCMS from a demo starter
  2. Open a blog post
  3. Click the history tab
  4. Click the 'JSON' link to get a response from the API

You will see the incorrect format

Acceptance Criteria

The format should be reverted to the previous date format unless the change was made for reason.

dotCMS Version

Tested on demo (23.05) and 23.01, this occurs in both.

It does NOT occur in 22.03 LTS, so the change occurred after then

Proposed Objective

Core Features

Proposed Priority

Priority 1 - Show Stopper

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response
@swicken-dotcms
Copy link
Contributor Author

Two tickets this impacts:

https://dotcms.zendesk.com/agent/tickets/111365
https://dotcms.zendesk.com/agent/tickets/111558

fabrizzio-dotCMS added a commit that referenced this issue May 23, 2023
@fabrizzio-dotCMS
#25008 adding a strategy to normilize date, date-time, and time field…
6fe3eb0 
…s read from the contetlet as json
@fabrizzio-dotCMS fabrizzio-dotCMS linked a pull request May 23, 2023 that will close this issue
Issue 25008 dateformat on content resource #25017
Merged
fabrizzio-dotCMS added a commit that referenced this issue May 23, 2023
@fabrizzio-dotCMS
#25008 adding test
09e2404
nollymar pushed a commit that referenced this issue May 26, 2023
@fabrizzio-dotCMS
Issue 25008 dateformat on content resource (#25017)
0bc51e3 
* #25008 adding a strategy to normilize date, date-time, and time fields read from the contetlet as json

* #25008  adding test
@nollymar
Copy link
Contributor

Internal QA: Passed

@nollymar nollymar removed their assignment May 26, 2023
jdotcms pushed a commit that referenced this issue May 26, 2023
@fabrizzio-dotCMS @jdotcms
Issue 25008 dateformat on content resource (#25017)
b7e5927 
* #25008 adding a strategy to normilize date, date-time, and time fields read from the contetlet as json

* #25008  adding test
@bryanboza
Copy link
Member

Fixed, tested on release-23.06 // Docker // FF

@erickgonzalez erickgonzalez added Next LTS Release LTS: Excluded Ticket that has been excluded from at least one LTS labels Jun 2, 2023
@erickgonzalez
Copy link
Contributor

Not reproducible in 22.03 LTS

erickgonzalez added a commit that referenced this issue Jun 7, 2023
@erickgonzalez
#25008 include in 23.01.4
38c6be8
@erickgonzalez erickgonzalez added the Release : 23.01.4 Included in LTS patch release 23.01.4 label Jun 7, 2023
@erickgonzalez erickgonzalez added Release : 22.03.7 Included in LTS patch release 22.03.7 and removed Release : 22.03.7 Included in LTS patch release 22.03.7 labels Jun 16, 2023
@erickgonzalez erickgonzalez mentioned this issue Jun 20, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
LTS: Excluded Ticket that has been excluded from at least one LTS OKR : Customer Success Owned by Arno OKR : Customer Support Owned by Scott QA : Approved QA : Passed Internal Release : 23.01.4 Included in LTS patch release 23.01.4 Release : 23.06 Team : Scout Type : Defect
Projects
dotCMS - Product Planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue 25008 dateformat on content resource dotCMS/core
6 participants
@bryanboza @erickgonzalez @nollymar @fabrizzio-dotCMS @swicken-dotcms @damen-dotcms