search

yap-zong-xin / pe

0 stars 0 forks source link

Accepting multiple Matriculation year to be edited #2

Open yap-zong-xin opened 4 months ago

yap-zong-xin commented 4 months ago

Description

Although only 1 matriculation year is accepted by the app, however, it allows user to edit more than 1 matriculation year without an error message.

Steps to reproduce

edit 1 my/2001 my/2015

Expected behaviour

There should be an error message of more than 1 my/ edited.

Actual behaviour

App accepted the command instead.

image.png

Reason for severity

Might caused confusion to the user and data inconsistent since user do not know which my was taken by the app.

nus-se-bot commented 4 months ago

Team's Response

Rejected as the correct command format provided both in the app as well as in the user guide has not been followed to produce this bug. (In both places, it is stated that only one my/ should be typed)

In addition, the edited person's details are stated in the message box upon editing the contact, as well as the PersonCard on the ui. It would then be impossible for the user to 'not know which my was taken by the app' as it is explicitly stated there.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]