-
Notifications
You must be signed in to change notification settings - Fork 108
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
RA-1988 - Allow DatetimePicker to support different timezones #102
base: master
Are you sure you want to change the base?
Conversation
…to "dd MMM yyyy HH:mm:ss" - Change datePicker format from "dd M yyyy HH:mm" to "dd M yyyy hh:ii:ss" (this is time used after picking with the datetimepicker) - Updating format every time a change happens (this is required in order to send timezone)
Hi @dkayiwa, |
@icrc-toliveira Thanks for working on this, Do you mind sharing the screenshot . |
- Added new parse format for start/end date to prevent known error - Changed datePickerFormat format to "dd M yyyy hh:ii:ss" (prevents parse error for french) - Dates are now always saved as "YYYY-MM-DDTHH:mm:ss.sssZ", and timezone "cleaning"is done by htmlformentryui
…commons into RA-1988 � Conflicts: � omod/src/main/webapp/fragments/field/datetimepicker.gsp
…ut requesting time, while keeping timezone - To accomplish that, date is parse into "DD MMM YYYY", while keeping the timezone in the input format
Hi @sherrif10 Sorry for my late response, When developed, datetimepicker didn´t seem to take time zones into account, and in order for this to work a lot of stuff had to be changed.
|
@dkayiwa could you kindly review this for merging? Apparently it's one of several blockers for ICRC. |
@icrc-toliveira remember to always hit the |
Issue: https://issues.openmrs.org/browse/RA-1988
Description:
When using a date time picker (with time), it will not store the date with time-zone information.
This will result in date being stored always as if was UTC, even when is not.
Changes done:
Related PR's: