AY2324S2-CS2103T-F13-2 / tp

https://nus-cs2103-ay2324s2.github.io/tp/
MIT License
0 stars 4 forks source link

[PE-D][Tester A] Rationale for sorting from old to new dates for Last Contacted is unclear #139

Closed nus-pe-bot closed 7 months ago

nus-pe-bot commented 8 months ago

Details

It is unclear why the command lastcontact sorts from the oldest to newest date using the field: Last contacted.

Usually in contact applications, such as on our mobile devices, the list of calls goes from the most recent to the oldest. As the list of clients gets large (>50 as specified by the user guide) for the intended target audience, it may be inconvenient for the financial advisor if the dates were sorted from the oldest to the most recent for last contact dates, as clients with very old last contact dates may be past clients that are no longer employing the financial advisor. Furthermore, clients with more recent Last contacted dates are more likely to be active, and hence, as a financial advisor, I may want to look at those profiles more often.

How to reproduce

  1. Launch the application
  2. Type in the command: lastcontact

Expectations

It may be better to sort Last contacted from newest to oldest date, like how our recent calls are sorted.

Actual

Annie is at the top of the list even though the user has not contacted her since 1997. When the client list gets large this may become a problem when "inactive client contacts crowd out the "active" ones.

image.png


Labels: severity.Medium type.FeatureFlaw original: J-wang-CSMA/ped#15