chanyijuan / pe

0 stars 0 forks source link

Error message wording inconsistent #7

Open chanyijuan opened 2 years ago

chanyijuan commented 2 years ago

Am nitpicking here, but wording of the command confirmation messages could be standardised to either active or passive phrasing.

Successful addClient:

Screenshot 2022-04-16 at 2.41.50 PM.png

Successful deleteProc:

Screenshot 2022-04-16 at 2.41.59 PM.png

Either 'Added client: {client details}' or 'Procedure deleted: {procedure details}' would make the application seem more polished.

soc-pe-bot commented 2 years ago

Team's Response

The success message can be interpreted clearly and the phrasing has no issue. This would rather be a subjective issue on English styling, which is not an examinable topic under the PE.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[ ] I disagree

Reason for disagreement: [replace this with your explanation]