search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Multiple planned enhancements affect multiple features #439

Open nus-pe-bot opened 6 days ago

nus-pe-bot commented 6 days ago

Multiple enhancements mentioned by the group cover more than 1 feature.

As mentioned by the course website, one enhancement should cover one feature.

image.png

Confirmed with prof as well

image.png

As such, from this bug report onwards, only consider the following

image.png

Above is 2 features

image.png Above is 6 features (due to mentions of the 3 add, edit and delete for both entities)

image.png Above is 2 features

Totals to 10 planned enhancements based on group size of 5.

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

Jayden-Kim-NUS commented 6 days ago

Team's Response

Thanks for submitting this report.

Comments from dev team: image.png

Planned Enhancement 2: In our app, the add, edit, and delete commands for person and events have superclasses that encapsulate the command without specification of person or event (addcommand, editcommand, deletecommand). Implementing the autoswitch for add p will also implement it for add e automatically as the logic for switching will be contained in the superclass, ie it is not possible to add it for persons only without rewriting a huge chunk of code as the commands are objects from the same class. Thus we would count this as 3 (and not 6) planned enhancements.

That would mean planned enhancements 1 to 6 have immunity.

Bug Type that we chose: same

Severity that we chose: same

Decision that we made: Accept We hope that you find our response reasonable.

Duplicate status (if any):

--