search

0-yibai / pe

0 stars 0 forks source link

Format for Deleting House Command in the User Guide might be wrong #5

Open 0-yibai opened 2 months ago

0-yibai commented 2 months ago

In the UG, deleteHouse marks Blk and level to be optional by putting square brackets around them.

Screenshot 2024-04-19 at 17.12.19.png

Screenshot 2024-04-19 at 17.12.41.png

However, the actual app does not allow leaving out these 2 areas.

Steps to reproduce: deleteHouse n/Bavid Li type/Hdb street/Clementi Ave 2 postal/578578 price/10000 unitNo/25

Expected Behaviour according to UG: house deleted successfully

Actual Behaviour: Error: Hdb must have a block and a level! Screenshot 2024-04-19 at 17.10.02.png

I would assume that this is a documentation bug because block and level seem necessary for the identification of a house.

nus-pe-bot commented 2 months ago

Team's Response

We have specified that "The exact House restrictions applied in addHouse apply here too." under deleteHouse command. Kindly refer to the attached images. image.png

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]