search

ramapriyan912001 / pe

0 stars 0 forks source link

Used emojis in company name (such companies exist nowadays) and incorrect error message #5

Open ramapriyan912001 opened 2 years ago

ramapriyan912001 commented 2 years ago

From this command (see video, not sure if it's crystal clear but i used the thumbs up emoji 👍), you see if i type an emoji into the company part, I get the wrong error message as my command format is in fact correct.

Video_21-11-12_16-41-41.mp4

nus-pe-bot commented 2 years ago

Team's Response

Hi. Thanks for the review and the screen recording. Based on your video, it seems as if there is a white space between the emoji and p/ although that is actually not the case (you can try to reproduce it). Similar to the issue you reported, successive fields need to be separated by a whitespace, so typing add c/(emoji)p/... would be interpreted as an attempt to add a company with name (emoji)p/....

This results in the command lacking a p/, which is a necessary field for a valid command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]