moziliar/pe

pe's Issues

Manual Testing bug

It was written that
"b. Re-launch the app by double-clicking the jar file."
in relaunch step.

Expected:
The jar file opens

What happened:
jar file fails to open (same as Duke when double clicking it)

Email and its changes not reflected in interviewer tab

Expected:
Email is shown in the interviewer entry and any changes to it (changing it to [email protected]) should be reflected (meaning the interviewr has been edited)

What happened:
Command box gives correct feedback. However, the changes to the email is not reflected in the entry and hence unable to identify if the entry is indeed edited.

UI scrolling user-unfriendly

The scrolling pattern of schedules does not give a good user experience. Scrolling in the same subpanel downwards will lead to rightward scroll before hitting the right end. The scrolling continues and it happends again in the second subpanel. Lack of CLI commands to move around the schedules tab to pin point allocated slots.

Allocated slot not reflected after successfully scheduling an interview slot

Expected:
After executing schedule command and receiving the feedback that some interviewees are allocated an interview slot (from here we inter that Barry is allocated a slot), allocated slot field of Barry should change.

What happened:
Barry's allocated slot field remains the same as Jim who is not allocated a slot.

Removing available slots for interviewer not allowed

It seems that empty available slot entry for an interviewer is allowed. This is ok given that in real life some interviewers are indeed not free. So as an interviewer who just get other commitment that overlaps with the available slot, I would want to remove my available slot so that no interview can be scheduled during that slot (in fact I am not available).

Expected:
By using edit command, edit Person C r/interviewer s/[Empty] allows a user to remove the interview slot

What happened:
This is disallowed. In this case, if the user is not free anymore, he/she would either need to delete his/her entry or have no choice but to accept the interview that might be scheduled then (since he/she is technically "free" in the app). This is user unfriendly as the user might want to keep his/her entry for convenience sake while emptying up the available slots field (since it is allowed to be empty).

Clear gives no prior warning before purging the entire app

As a simple command, clear purges all data in the app as an irreversible process. This might give user trouble
if the user accidentally execute the command with a lot of important info in the app stored.

Unable to email to an interviewee to whom the emailing process was aborted before via email all feature

Context:
Tried emailing to Barry (with interview slot) but aborted (as the email is incomplete) and run email ct/alltimeslot command

Expected:
Since Barry is not emailed (not sure how is it indicated), email all will attempt to email to Barry again.

What happened:
Failed to email Barry as it is indicated that Barry has been emailed before.

Bug:
The user should be able to abort sending of the email and somehow get to resend to the interviewee via either email by name or email all. However, this is not the case. Lack of inidicator to show that a person is emailed.