nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Reservation is not logical #5583

Open nus-pe-bot opened 1 month ago

nus-pe-bot commented 1 month ago

Currently, if the I want to take notes about reservations, I should use the index of contacts which contains not only the name and phone number but also the email and address to add reservations. However, it is very strange that the restaurant owners have the customer's real address and email address. Also, they may not save all the customers' contact information privately. Thus, it would be better

  1. make address and email optional(or maybe not optional only for employee and supplier)
  2. can use "rsv" command with name, phone number, d, t, p, like add instead of using the already saved contacts by using index.

[original: nus-cs2103-AY2324S2/pe-interim#5576] [original labels: severity.Medium type.FeatureFlaw]

ziiqii commented 4 weeks ago

Team's Response

This was already captured in our planned enhancements.

image.png

Having less compulsory fields makes it much easier to add a contact. This in turn makes it easier to make a reservation for a contact, without requiring the customer's real address and email address like the reviewer has suggested.

Reviewer's second suggestion is a fundamental change to the functionality of the rsv command, and is not relevant to our feature.

Duplicate status (if any):

--