search

nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

After cancelling leave, the worker should go back to work. #180

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

After cancelling leave, the worker should go back to work. But now, it is only to cancel all leaves and release all work assignments. Is it valid?

image.png

[original: nus-cs2103-AY2021S1/pe-interim#164]

wakululuu commented 3 years ago

Team's Response

This is the intended behaviour since after a worker has taken leave for a shift, the manager would look for a replacement to stand in for the worker on leave. So even after that worker cancels their leave later on, the worker would no longer be needed for their previously assigned shifts as there would already be replacements filling up their position. We believe it is more intuitive for users to add the worker back than to have to unassign them when the previous assignment is no longer what the manager wants in the system.

Furthermore, according to the automated message this bug report was unfortunately made outside of the allocated time in the PE for Phase 1.

Duplicate status (if any):

--