johnnythesnake12 / pe

0 stars 0 forks source link

leaves with overlapping timelines are allowed #8

Open johnnythesnake12 opened 11 months ago

johnnythesnake12 commented 11 months ago

image.png

If want to apply for leaves, overlapping timelines should not be allowed. in this case, there are 2 leaves that overlap on november 1st to 2nd for the same user

nus-pe-script commented 11 months ago

Team's Response

There are scenarios in which an HR manager may want to keep a record of overlapping leaves. For example, an employee might have requested leave to go on a holiday. However, the employee suffered an accident during the holiday and was hospitalised. In a scenario like this, the remaining period of the original leave should be converted into medical leave, thus requiring a separate leave application. However, in the interest of bookkeeping, the HR manager might want to retain a record of the original leave applied. Thus, there is a need to allow overlapping leaves.

On the other hand, we do not allow leaves with duplicate periods as it is more likely that the HR manager is inputting duplicated leave applications rather than a similar scenario as the one described above.

However, we understand your concern that there should be additional safeguards against inputting concurrent leaves - we will look into implementing warnings to highlight potential overlaps.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]