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.

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

Data Importer does not set time properly #3437

Open
3 tasks
EAVilleda opened this issue Sep 26, 2024 · 0 comments
Open
3 tasks

Data Importer does not set time properly #3437

EAVilleda opened this issue Sep 26, 2024 · 0 comments

Comments

@EAVilleda
Copy link
Contributor

Required Information

  • AEM Version 2024.6.16799.20240614T073455Z-240500
  • ACS AEM Commons Version: 6.6.2
  • Reproducible on Latest? N/A (cannot check at this time)

Expected Behavior

I am trying to use the Data Importer to create pages with date properties. Going by what is mention in the documentation, I assumed "A representation of date and time" should work.

Actual Behavior

The dates are properly being set, but the time is set to midnight. I have the column header set as "jcr:content/eventTime@date". I have tried entering the data "2024-10-15T17:00:00.000-05:00" as a General cell, "2024-09-15 17:00 +00:00" as General cell, and "2024-09-15 17:00 +00:00" as Date cell. The time is always zeroes, such as "2024-09-15T00:00:00.000-05:00".

  • Am I using the wrong format to get time saved in the property?
  • Was the wording misleading, and only date can be imported?

If time is not meant to be imported (so it would be a feature request), I can maybe work on this in the future. I just want to be sure my issue is not caused by user input.

Steps to Reproduce

  1. Create a spreadsheet to create a new page, including a column for date property such as "jcr:content/eventTime@date".
  2. Use the Data Importer, be sure to include jcr:content nodes.
  3. Uncheck dry run and start.
  4. Confirm page was created.
  5. Check the date property. It should have the entered timestamp. (If the dialog field does not show both date and time, check in CRXDE.)

Links

I unfortunately do not have time to provide a package using we-retail pages. But hopefully this can be simply answered rather than need testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant