ishan-agarwal-05 / pe

0 stars 0 forks source link

Duplicate detection in radd is case sensitive #9

Open ishan-agarwal-05 opened 4 days ago

ishan-agarwal-05 commented 4 days ago

The follwong commands

radd 6 a/ 201 BukiT Batok Street 21 #10-163 s/ 01/01/2024 m/3200 and radd 6 a/ 201 Bukit Batok Street 21 #10-163 s/ 01/01/2024 m/3200

both should logically give the same property but are treated as 2 different ones.

Screenshot 2024-11-15 at 5.29.44 PM.png

nus-pe-script commented 1 day ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

According to the User Guide, it has already been stated that TrueRental handles duplicate detection based on “rental information is only considered duplicate if and only if the value of ADDRESS parameter matches exactly, including case sensitivity”.

This is to allow the user to have the flexibility in manipulating the addresses.

image.png

However, we believe it can be better, hence we will classify it as NOT-IN-SCOPE. In addition, it is less important than the work done in the current version of the product e.g., it was not related to features delivered in v1.6 or lower priority than the work already done in v1.6.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]


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