nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

View Company command does not handle case sensitivity #2248

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

Command add company handles case sensitivity.

For example: add job n/Software Engineer c/google s/100000 d/Looking for an exceptional individual is successful as shown in the image. image.png

Next, we call view company 1 . This only shows the first job, as shown in the image below. There is an inconsistency between add company and view company

add company allows a job from the company from "google" to be added and handles case sensitivity. However, view product does not show jobs from "google". This can be misleading to the user causing the user to miss out out potential jobs offered by the company.

image.png

Similarly, if a contact is assigned with the job from "google", as can be seen from the image below:

image.png

view_company 1 will not display the contact.

image.png


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

KengHian commented 1 week ago

Team's Response

Good catch!

Duplicate status (if any):

--