nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Present time reference not indicated in DG or UG #58

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

One of the interesting features of the app is that it can take into account current time, and compare it to the user's input of time.

However, it is not indicated in the DG or UG where the 'present time' comes from. It is unclear if the present time reference comes from the internet, or the device time, or if the app manages its own clock


[original: nus-cs2103-AY2425S1/pe-interim#77] [original labels: severity.Low type.DocumentationBug]

yugan01 commented 1 week ago

Team's Response

While it is slightly beneficial to the reader if we clearly stated what present time means, it is implicitly understood that present time refers to the time zone that you are in. So there is no explicit need to state where the 'present time' comes from. Furthermore, for the DG, it is an overview of how the code works. Hence, the implementation of present time will be excessively detailed and not suitable to be included in the DG.

Duplicate status (if any):

--