MSFREACH / msf-reach

Web platform for MSF-REACH
https://msf-reach.org/
4 stars 5 forks source link

Dates #633

Closed matthewberryman closed 6 years ago

matthewberryman commented 6 years ago

Can we take a look to ensure that all dates entered are being stored correctly and displayed correctly, especially on Safari (which doesn't always handle ISO 8601 properly, dependent on version).

mehrdadgit commented 6 years ago

This was both chrome and safari issue.

I fixed the date not saving/ showing correctly in the above commit for mission modal. There is still an issue that in edit-mode <date-picker> does not prefill with the current datetime!? still looking into it... . fixed now for departure and arrival times in the commit below.

@qclin I've ran out of time today; could you please double check all other dates in modals/event edits etc and make sure that date save/display functionality is working as it should ? (Note in DB sometimes dates are stored in different formats e.g. sometimes in 'DD/MM/YYYY' and sometimes as a full ISO string; I have handled both cases in my above commit (compare formatDateOnly and formatFullDate)

Plus note the trick in commit below when you want to capture time as well.

Also (re https://github.com/MSFREACH/msf-reach/commit/b7b04d7fb926d9f537490d586a625a0ed480734e ) @qclin is there an easy way to update landing page without hard refresh on mission edits? if not could we do the page refresh just when the modal is closed; it is very annoying at its current state.

qclin commented 6 years ago

@mehrdadgit you can close the modal and call load mission table again

matthewberryman commented 6 years ago

@mehrdadgit @qclin Just noting this issue applies to all dates not those on missions modal. There are a number of issues still:

matthewberryman commented 6 years ago

Also opened date invalid

screenshot 2018-09-10 09 59 08
matthewberryman commented 6 years ago

Just noting since fix for #635 and #632 removed event closure date prompt, we probably need to prompt for that on event completion status change in event editing panel.

matthewberryman commented 6 years ago

Re-opening to track display of mission end, if present, otherwise close date, properly in table.

matthewberryman commented 6 years ago

@LucieGueuning ready for review in test environment

LucieGueuning commented 6 years ago

wording should be (for consistency):

matthewberryman commented 6 years ago

wording changes made in 212ef98