nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Sorting of numerical names #2216

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

Feature Flaw (Severity LOW) Steps to reproduce

  1. Add people with only numerical names
  2. Use the sorting function to get sort by field name

Expected for name 2 to be sorted ahead of name 12345678 Actual contact 2 is under contact 12345678 Screenshot 2024-04-19 at 4.36.32 PM.png


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

yleeyilin commented 4 months ago

Team's Response

In the UG, we specified clearly that the field is sorted in lexicographical order, not numerical ordering. Thus, this is a correct behaviour as intended.

telegram-cloud-photo-size-5-6138804929951809370-x.jpg/var/folders/tg/yrcwyrrj2_qc1tf0swypgctw0000gn/T/ru.keepcoder.Telegram/telegram-cloud-photo-size-5-6138804929951809370-x.jpg

Duplicate status (if any):

--