search

jarrellc1 / pe

0 stars 0 forks source link

Capital error for work experience when adding #5

Open jarrellc1 opened 1 week ago

jarrellc1 commented 1 week ago

Although UG mentions that first letter should be caps, the error message could be more specific to mention the type of error. FE7B531A-FC07-411B-B64F-2C46684D6DF9.jpeg Right now it only says this which is not very specific.

nus-se-bot commented 3 days ago

Team's Response

We already mentioned in UG that inputs should be capitalised, as well as in planned enhancements that this will be changed

The 'Original' Bug

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

Add work command: Unexpected error message

steps: addw in/8 w/intern,google,2024

expected: the error message should inform the user that the input for w/ is wrong

actual: the error message states that the command format is wrong when it is not, which leads to user confusion. since this is a main feature and users should not be expected to memorise the details of the input format, this will be frustrating for users.

image.png

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

Their Response to the 'Original' Bug

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

image.png

image.png

We have stated in the UG and planned enhancements section in DG that this will be improved in the future, and that inputs should be capitalised.

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: [replace this with your explanation]

## :question: Issue response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]