hansneddyanto / pe

0 stars 0 forks source link

Present time reference not indicated in DG or UG #8

Open hansneddyanto opened 4 days ago

hansneddyanto commented 4 days 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

nus-pe-bot commented 15 hours ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

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.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]