bryanhce / pe

0 stars 0 forks source link

Unable to set availability with Professors #14

Open bryanhce opened 1 year ago

bryanhce commented 1 year ago

Screenshot 2022-11-11 at 4.50.36 PM.png

This is a feature flaw as Professors do conduct consultations with students and there is no feature for the professors to indicate their availability. There are also some classes such as MA1521 AY21/22 Sem1 where the professor functions as the TA for the module. He would then not be able to show his availability on the application.

nus-pe-script commented 1 year ago

Team's Response

Our app is made for TAs and Professors to use. The user story for availability is down below

Screenshot 2022-11-14 at 6.02.01 PM.png

As such, there would be no need for professors to have availabilities.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I feel like your team's response did not give more clarity about this bug or properly addressed the issue raised. As previously mentioned, your application lacks the ability for professors to set up consultations by having an availability. This has nothing to do with viewing the availability of the TAs as mentioned in your response.

The missing feature for professors to have availability should be a feature flaw as it fulfils the requirements as seen in the screenshot below: the first point about missing feature and the second point, sub point 2 about better implementation without much additional effort as this behaviour is already implemented for TAs and should be easily extended to Profs.

Screenshot 2022-11-15 at 2.02.25 PM.png