nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Inconsistent command format for addtoconsult and delete #3414

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

addtoconsult uses i/ while delete uses ; to list out indices

This can be very confusing to use as only certain commands are allowed these shortcuts.


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

S-K-Y-Light commented 1 week ago

Team's Response

According to https://nus-cs2103-ay2425s1.github.io/website/admin/tp-pe.html#functionality-bugs-2

image.png

Refer to Last Line. If the user attempts to use semicolon for commands that do not support as such, TAHub will display an error message informing you.

Our Team decided not to have a semicolon for all commands as this is not as important as the other features, and it doesn't affect the functionality of TAHub.

Duplicate status (if any):

--