Open-event-droidgen: Delegate Date string formatting to a single DateUtil class

Created on 25 May 2017  路  7Comments  路  Source: fossasia/open-event-droidgen

Actual Behaviour

Currently, we are manually parsing dates in various sections of app, where some ignore timezone and some use another mechanism to parse it.

Expected Behaviour

For an event-based application, it is trivial to do correct and standard date conversions so it must happen in a single tested class which correctly handles all edge cases and TZ information correctly

Would you like to work on the issue?

Yes, but if anyone else is interested, he/she may take it up

Most helpful comment

18789138_10211406595515063_371526646_o

@iamareebjamal should i make any changes?

All 7 comments

@arp95 The current date handling is very erroneous and repetitive. I am rewriting the date class, so you can hold the test writing for now. Sorry for the inconvenience

@iamareebjamal can i work on that? i actually made changes so would like to work on that.

Definitely. Thank you

Please share the proposed API before writing the code here and is Milestone 4 OK?

yeah its fine.
will share the functions i am using and then will submit the pr. is it fine?

Yes :+1:

18789138_10211406595515063_371526646_o

@iamareebjamal should i make any changes?

No, it's great :+1:

Was this page helpful?
0 / 5 - 0 ratings

Related issues

dr0pdb picture dr0pdb  路  7Comments

97k picture 97k  路  5Comments

iamareebjamal picture iamareebjamal  路  8Comments

mahikaw picture mahikaw  路  7Comments

mananwason picture mananwason  路  9Comments