nighoggDatatype / pe

0 stars 0 forks source link

Unclear how the late fee is calculated #2

Open nighoggDatatype opened 3 years ago

nighoggDatatype commented 3 years ago

It is unclear what the late return fee is if a book is returned late, or how it is calculated. I rate it of high severity since money matters should not be left clear, and this is a likely cause for disputes.

nus-pe-bot commented 3 years ago

Team's Response

No response provided.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

It is not possible to set any hourly rate for late fees

I currently assume that the late fees are calculated on a hourly basis based on this screenshot:

sTU92NaoCK.png

Currently, there is no way to set a custom rate via a stand alone function.

SumatraPDF_vNcUgY5m45.png

Even if there is, as a hidden feature in one of the config files, it is not stated in the User Guide, and if it is in one of the other commands, then it would be placed wildly inappropriately.

I rate this of high importance since this is a matter of money, which might make it useless to determine late fees.


[original: nus-cs2103-AY2021S2/pe-interim#1974] [original labels: severity.High type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

This is not a bug. We will be engaging our client with regards to the cost of overdue books, which will be predetermined by our very first contact with the client.

By preventing the client from being able to access the cost of each book, this would also help to prevent the occurrence of unnecessary complications in the borrowing process, e.g. the client setting a wrong price for a book, or accidentally deleting the overdue cost of a book.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.DocumentationBug]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: [replace this with your reason]