nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Meetings not in chronological order #1187

Open nus-se-script opened 2 months ago

nus-se-script commented 2 months ago

Describe the bug The meetings are not displayed in chronological order

To Reproduce Steps to reproduce the behavior:

  1. Add an appointment
  2. Add another appointment with an earlier date
  3. Notice that the meetings are added based on order of addition, not based on the appointment timing

Expected behavior More likely than not, end users will want to be able to view the meetings based on chronological order (e.g most recent to least recent)

Screenshots Screenshot 2024-04-19 at 4.32.53 PM.png

Additional context Nil


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

rahhulleee commented 2 months ago

Team's Response

We have a separate "Meetings" command that provides the chronological ordering for upcoming meetings in the week. This is specified in our UG.

We didn't sort the meetings section so that the index of the meeting corresponds to index of the person with the meeting. This way the meeting and the person with the meeting would be side by side and it's easy to see.

Duplicate status (if any):

--