nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Add duplicate finances #3768

Open nus-se-script opened 8 months ago

nus-se-script commented 8 months ago

Adding duplicate expenses/commissions is allowed, which should not be the case and the UI should warn the user.


[original: nus-cs2103-AY2324S1/pe-interim#5197] [original labels: type.FeatureFlaw severity.VeryLow]

flexibo commented 8 months ago

Team's Response

Thanks for the feedback! However, we have will have to disagree with you on this.

Under both the add-c and add-e sections of our UG we have mentioned that we will allow for duplicate finances to be added. This is because it is not uncommon for freelancers to have a duplicate finance entry. For example, if a freelancer is receives two commissions from Company A for two projects on the same day and wishes to save them as add-c a/500 c/Alex Yeoh d/Company A t/10-10-2023 , add-c a/500 c/Alex Yeoh d/Company A t/10-10-2023 these should be valid inputs.

Additionally, a freelancer can have two expenses on the same day for the same items add-e a/500 d/Pudding t/10-10-2023 , add-e a/500 d/Pudding t/10-10-2023. These are valid expense inputs as well.

image.png

image.png

Duplicate status (if any):

--