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

Invalid time on trip #498

Closed
stephhuerre opened this issue Feb 3, 2020 · 2 comments
Closed

Invalid time on trip #498

stephhuerre opened this issue Feb 3, 2020 · 2 comments

Comments

@stephhuerre
Copy link

Hi @shankari,

Our trips from January 30th and 31st had an undefined date Invalid dateHundefined.

After some investigation we found that the month value of tripgj.data.properties.start_local_dt received from the database was equal to 1 (which corresponds to February). Since there are only 29 days in February, the 30th and 31th return an erreur.

How should this appropriately be fixed on the server side ?

In the mean time, we fixed it by replacing tripgj.data.properties.start_local_dt with tripgj.data.properties.start_fmt_time in list.js, line 210 and 211.

Thanks!

@stephhuerre stephhuerre changed the title Invalide date on trip Invalid time on trip Feb 3, 2020
@shankari
Copy link
Contributor

shankari commented Feb 3, 2020

@stephhuerre there is an open PR to fix this (e-mission/e-mission-phone#615) in which I suggested the principled fix (we should keep the server untouched and fix the client, so that both draft trips and processed trips can be fixed). Unfotunately, @atton16 has not yet updated the PR. If you want to implement a Right Fix and submit it to master, that would be great!

@shankari
Copy link
Contributor

shankari commented Aug 8, 2020

Right Fix checked in 8537d9d4d05d7361fe8cf9ea6871411abd962f43
Thanks to Transway for the contribution.

@shankari shankari closed this as completed Aug 8, 2020
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

2 participants