search

Austintjh19 / pe

0 stars 0 forks source link

Allowing duplicated entries of Company Names - Whitespaces are not Ignored #5

Open Austintjh19 opened 1 month ago

Austintjh19 commented 1 month ago

image.png

The application allows duplicated entries of Company Names, so long as it does not match character for character. Inclusive of whitespaces.

NUS HUB and NUS HUB should be the same company.

soc-pe-bot commented 3 weeks ago

Team's Response

Both talks about the allowance of duplicate entries within the start up names.

The 'Original' Bug

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

Allowing Duplicate Entries of Same Start Up Names (E.g. Google and google)

When adding the startup with name google, this is allowed by the application. This does not makes sense as they should be the same company. As shown below:

image.png

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

Their Response to the 'Original' Bug

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

This has already been addressed in the UG, directly under the section with add with the exact same example.

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.NotInScope`] - [x] I disagree **Reason for disagreement:** It is a feature flaw in my opinion, makes your CapitalConnect application less useful. Your find is case-insensitive shouldn't your add be too.