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

Fix format of time in user note #24529

Merged
merged 1 commit into from Apr 23, 2019

Conversation

Projects
None yet
7 participants
@chmst
Copy link
Contributor

commented Apr 8, 2019

Pull Request for Issue #24462.

Summary of Changes

Add filter utc to the input field.
If the date is entered in for example german, the date in the input field has the format i.e. 21.3.2019 which is not integer and therefor is reset to Nulldate.

Testing Instructions

Use a backend language which does not use the YYYY-MM-DD format, for example German.
Add a date into the chek date.

Expected result

The date is stored.

Actual result

The date is empty

Documentation Changes Required

@chmst chmst changed the title Fix format of time Fix format of time in user note Apr 8, 2019

@alikon

This comment has been minimized.

Copy link
Contributor

commented Apr 8, 2019

I have tested this item successfully on 03bd0ad


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24529.

1 similar comment
@ChristineWk

This comment has been minimized.

Copy link

commented Apr 8, 2019

I have tested this item successfully on 03bd0ad


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24529.

@Quy

This comment has been minimized.

Copy link
Contributor

commented Apr 8, 2019

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24529.

@joomla-cms-bot joomla-cms-bot added the RTC label Apr 8, 2019

@HLeithner HLeithner merged commit abdf683 into joomla:staging Apr 23, 2019

5 checks passed

Hound No violations found. Woof!
JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/drone/pr the build was successful
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@HLeithner

This comment has been minimized.

Copy link
Member

commented Apr 23, 2019

thx

@joomla-cms-bot joomla-cms-bot removed the RTC label Apr 23, 2019

@HLeithner HLeithner added this to the Joomla 3.9.6 milestone Apr 23, 2019

tecpromotion added a commit to tecpromotion/joomla-cms that referenced this pull request May 23, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.