AY2122S2-CS2103-F11-2 / tp

TAlent Assistant™ is a desktop application used by professors to schedule interviews and keep track of applications for teaching assistants.
https://ay2122s2-cs2103-f11-2.github.io/tp/
MIT License
1 stars 5 forks source link

[PE-D] Details about clashing interview not specified #336

Closed nus-pe-script closed 2 years ago

nus-pe-script commented 2 years ago

When adding a new interview that clashes with an existing one, the error-message is displayed:

Screenshot 2022-04-01 at 3.44.12 PM.png

However, the user is not informed as to which interview clashes with this one, and has to go down the list in the UI to find the index of the interview in order to find out more details about it and maybe delete it.

User has to manually go through the list of scheduled interviews to find the interview that clashes with the one that is being added:

Screenshot 2022-04-01 at 3.44.28 PM.png

Personally, I feel that this is not a big issue if the number of scheduled interviews is low as the user can quickly glance through the list of interviews to find the conflicting one.

However, if an extremely-popz prof were to use the program, he may have numerous scheduled interviews, which would make finding the conflicting interview slightly more onerous.

Suggestion

In such a situation, maybe the list of scheduled interviews could be refreshed to show only the interview that conflicts with the one that the user is trying to add.

This way, the user can get immediate feedback without any extra work on his/her part!


Labels: severity.Low type.FeatureFlaw original: jamesyeap/ped#10

domlimm commented 2 years ago

If have conflicting interview, error will pop up. Coming soon... (?)