nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Allow shift length to be variable #26

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

It is unlikely that in practice, all shift lengths are fixed at 8 hours. If this is not the case, then the worker-pay feature is no longer useful to shift managers.

Consider adding a feature to vary the shift length for each worker. This could be done by storing a worker's shift status as a ShiftAssignment field that stores the worker, shift time, role, and the length of the shift


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

wakululuu commented 3 years ago

Team's Response

A good addition to have but for our scope we limit ourselves to fixed shift hours. It is also mentioned in the UG that we have decided to fix our AM and PM shifts to 8 hours long.

The severity is lowered from Medium to Low as fixed shift length only affects worker-pay for the current iteration and the impact to the user is minimal as the pay calculation is shown to the user so the user is aware that 8 hours shift length is used for the pay calculation

Screenshot 2020-11-14 at 4.35.59 PM.png

Duplicate status (if any):

--