insafhere / pe

0 stars 0 forks source link

incorrect parameters are not reflected properly #11

Open insafhere opened 1 year ago

insafhere commented 1 year ago

image.png

over here code and module are incorrectly spelt, supposed to be "module" and "code" but I put them as "mod" and "cod" respectively but only the parameter for the module is shown as missing. perhaps, can reflect all the issues at once instead for better clarity.

like :

nus-pe-bot commented 1 year ago

Team's Response

Error messages are shown to help the user fix his/her command incrementally. This is by design to avoid swamping a user with information (think of c++ compilation error messages).

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Multiple errors in command

In the event that there are multiple errors with the command, the group can consider displaying all invalid error messages so that the user needed not have to run two commands to find out that there are two errors.

image.png


[original: nus-cs2113-AY2223S1/pe-interim#128] [original labels: severity.VeryLow type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

The app is designed to help users fix their commands incrementally rather than dumping many errors at the user.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: This is a separate and a completely different issue addressing the incorrect spelling of parameters.


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I believe that firstly my issue has been wrongly tagged to be a duplicate of a complete different issue. Secondly, it is an addressable concern. I believe the issue should be addressed.

In my example, code and module are incorrectly spelt, supposed to be "module" and "code" but I put them as "mod" and "cod" respectively but only the parameter for the module is shown as missing. Perhaps, can reflect all the issues at once instead for better clarity like :


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.High]

Reason for disagreement: It is an Addressable concern and should be placed under "Low" Minimally.