search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Command being too long due to too much compulsory fields #2452

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

add n/NAME p/PHONE_NUMBER e/EMAIL_ADDRESS r/ROLE et/EMPLOYMENT_TYPE s/EXPECTED_SALARY l/LEVEL_OF_EDUCATION y/YEARS_OF_EXPERIENCE [t/TAG] [i/INTERVIEW][nt/NOTES]

image.png

Most of the fields are not optional. This app is for all recruiters and it is restrictive. It is not rare that companies do not require some information. For instance, some recruiters may not have email as requirement when recruiting older people. Sometimes, Level of Education or expected salary may not be provided by the applicants.

By doing so, you are forcing users to type in some value in order to add an applicant.

[original: nus-cs2103-AY2122S1/pe-interim#2361] [original labels: severity.Low type.FeatureFlaw]

twothicc commented 3 years ago

Team's Response

Agreed. Some of these fields should be made to be optional.

Duplicate status (if any):

--