search

nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Explanation of boundary valid date input of delete_rec_session #3125

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

It does not describe the scenario where the boundary of valid date input i.e. the start and end of the recurring session is deleted, the original recurring session would not be split, instead the corresponding start/end session will be removed. The lack of explanation of the scenario may cause different expecation from user

[original: nus-cs2103-AY2021S2/pe-interim#3125] [original labels: severity.VeryLow type.DocumentationBug]

chwoozy commented 3 years ago

Team's Response

As explained from the description of delete_rec_session:

image.png

We can see that the split would result in two recurring sessions that span the period exclusively before and after the deleted single session. In this case, when we remove the start of the recurring session, the explanation is still clear that there will be a split, but the first half of the session will be non-existent as there are no recurring sessions exclusively before the deleted session date. Hope this is clear, thanks!

Duplicate status (if any):

--