leongdl135 / pe

0 stars 0 forks source link

`sort time a` command not working #4

Open leongdl135 opened 1 year ago

leongdl135 commented 1 year ago

Steps to reproduce

  1. Add random times to other jobs
  2. Type the command sort time a

Expected:

I would expect that the time would be sorted accordingly, however, lets take a look at index 4, the time has not been sorted properly. The next event is in 2022, however, events in 2023 are prioritized above it. This is dangerous for users who might miss their assignments since they did not see the time properly.

Outcome:

image.png

nus-pe-bot commented 1 year ago

Team's Response

This command falls under the sort time command, fixing the sort time command will fix both of these bugs by ascending and descending.

The 'Original' Bug

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

sort time d not working

Steps to reproduce:

  1. Type sort time d

Expected: Sorted by latest to earliest.

However,

image.png

As we can see, tasks in 2022 are shown before tasks in 2023.


[original: nus-cs2103-AY2223S1/pe-interim#2864] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

No details provided by team.

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]