nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Reminder function does not filter out applications that are overdue #5673

Open nus-se-script opened 10 months ago

nus-se-script commented 10 months ago
  1. User inputs remind --earliest

App sorts applications based on deadline, in ascending order. Deadlines that have passed are still shown, and are no longer 'upcoming deadlines'.

The UG states that this feature Reminds you of upcoming deadlines for job applications, so applications with deadlines that have passed are no longer important, and could be filtered out to minimise clutter.

image.png

As in the picture, the first 3 deadline have passed and are no longer 'upcoming' nor are they 'high priority'.


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

wamps-jp commented 10 months ago

Team's Response

You are correct in that remind --earliest still includes expired deadlines.

This could be solved in two ways:

  1. Changing the section that says Reminds you of upcoming deadlines for job applications in the UG (in which case this would be a documentation bug).
  1. Adding some kind of complete signifier when sorting by deadline.

Thanks!

Duplicate status (if any):

--