nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Pet duplication check not case senstitive #216

Open nus-se-script opened 1 day ago

nus-se-script commented 1 day ago

Issue

Pets are considered duplicates when they have matching name, species, breed, age and sex. However pets can be added and edited to have the same value just in different capitalisation. By intuition capitalisation of species and breed should all still be considered the same

image.png

How to replicate

Have a valid pet added into the application Add another pet using the same arguments given for the first pet, however change the capitalisation of its breed or species

For example first enter pet n/Megatron s/Cat b/Siamese a/3 x/M t/playful on a clear application then enter pet n/Megatron s/cat b/Siamese a/3 x/M t/playful


[original: nus-cs2103-AY2425S1/pe-interim#306] [original labels: severity.Low type.FunctionalityBug]

jloh02 commented 6 hours ago

Team's Response

Thanks for the feature request. Although, we do plan to support duplicate pets in future, which would render this feature fix useless.

image.png

Duplicate status (if any):

--