search

SherChew27 / pe

0 stars 0 forks source link

Same command name #5

Open SherChew27 opened 1 week ago

SherChew27 commented 1 week ago

These 2 commands have the same command word, and thus should be included under one command e.g. 2.5 deleting a student

Screenshot 2024-11-15 at 4.48.16 PM.png

soc-pe-bot commented 1 week ago

Team's Response

It is not the same command, but overloaded for the user's convenience. The converse would be more damaging for the user guide, as it may confuse the user.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The format should be delete, with an optional name or index, as is presumably how the command is implemented. As a counter example, there isnt 5 different edit commands for each field that the user can edit. (edit by name, edit by phone number, edit by address).