yockcheng / pe

0 stars 0 forks source link

Cancellations of user stories in Developer Guide #7

Open yockcheng opened 1 week ago

yockcheng commented 1 week ago

Quite a handful of user stories are cancelled out with no explanation of what this means in the Developer Guide. If the user stories are no longer relevant they should be removed instead and if they are supposed to mean something else, this should be specified too to avoid confusion. Additionally, the unsorted user stories with varying priorities makes it hard for the reader to make sense of the user stories.

Screenshot 2024-11-15 at 5.17.21 PM.png

nus-se-bot commented 1 week ago

Team's Response

Believe that this is not a bug but a personal preference, as some may prefer to keep currently unimplemented user stories, potentially for future development. Also note that this was actually highlighted in our dry run (link below), resulting in them being cancelled out rather than left as is. We also believe that them being cancelled out should be a clear indication that they are not implemented currently. That being said, they could be sorted for better clarity.

https://github.com/AY2425S1-CS2103T-F14b-4/tp/issues/187

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Screenshot 2024-11-19 at 3.29.00 PM.png

Based on the course website, this bug issue does not satisfy either one of the necessary bullet points shown above for it to be classified as NotInScope, and is also not a grammar issue. Thus, while I agree that the severity should be lowered to VeryLow, I believe this bug issue should have been accepted instead.


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]