BryanL2303 / pe

0 stars 0 forks source link

Realodex does not accept clients with same name #2

Open BryanL2303 opened 2 months ago

BryanL2303 commented 2 months ago

It is noted in the UG that it is intentional for Realodex to not allow duplicate names into the system.

image.png

However, given that a real estate agent with many clients is quite likely to run into clients with the same first and last name this might become an issue for many users. This could be worked around by using the clients name as well as phone numbers and address altogether to check for duplicates rather than just the names.

Users can still currently work around this by entering clients full names or using their own nicknames for clients so it is not a high severity issue.

soc-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Realodex should allow multiple clients with the same name

Since Realodex is meant for real estate professionals to manage clients, I believe that the constraint of only having 1 client with a name (mentioned in the UG) is a feature flaw. For example: a user could have 2 clients with the name Roy Lim (an arbitrary common name).

As such, this implementation decision would inconvenience users, and I believe it is a feature flaw.

Relevant UG Screenshot

image.png


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

image.png

We will reject this bug as it been mentioned in planned enhancement that we intend to improve this aspect by checking more fields.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]