Add Optional Field - Githubissues

AY2425S1-CS2103T-T15-1 / tp

Other

0 stars 5 forks source link

Add Optional Field #92

Closed winstonlimjy closed 2 weeks ago

winstonlimjy commented 2 weeks ago

Changes: cadd

cadd: Either phone or email is accepted. Fixes #86
cadd: Both phone and email are accepted.
cadd: Display relevant messages based on the input. Fixes #90

cedit

cedit: Accept empty phone. Fixes #97
cedit: Accept empty email. Fixes #96
cedit: Do not accept an empty name. Fixes #94
cedit: Do not allow both empty phone + email. Fixes #95

UI

Modify Ui: Remove the $address from the client field. Fixes #85

JUnit Tests

Update cadd test cases: Modify current tests and add more tests. Fixes #91
Update credit test cases: Modify current tests and add more tests. Fixes #98