Skip to content
New issue

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

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

Already on GitHub? # to your account

text_datetime_timestamp as a repeater field not saving proper timestamp #330

Closed
tw2113 opened this issue May 23, 2015 · 4 comments
Closed

Comments

@tw2113
Copy link
Contributor

tw2113 commented May 23, 2015

It's saving the current time instead of the set time.

https://wordpress.org/support/topic/date-will-not-save-saves-today-instead?replies=4#post-6984530

@tw2113
Copy link
Contributor Author

tw2113 commented Nov 14, 2015

@jtsternberg have you looked into this one at all? Original reporter is still having issues and wondering if there's possible chance for some sort of fix for it.

@turegjorup
Copy link

I can confirm I'm seeing the same issue. WP 4.4 beta4, CMB2 2.1.2 on both Firefox and Chrome. I'm also only seeing it with a repeatable datetime field.

Looking through #446 there are some changes to the code taht fails, but I still don't think they properly account for repeating datetime fields.

@jtsternberg
Copy link
Member

This should be good in trunk. Please test.

@turegjorup
Copy link

Just tested with WP 4.4-beta4-35734, repeatable datetime fields now works as expected. Thank you for fixing it.

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

No branches or pull requests

3 participants