search

c-wenlong / pe

0 stars 0 forks source link

Unintuitive feature: Lend #10

Open c-wenlong opened 4 months ago

c-wenlong commented 4 months ago

Description

Lend should only manage you lending money from others. It is not intuitive to have a negative number when talking about money (for a lay man). We don't say I lend you -$5 to represent that I owe you $5. Hence, I find it rather unintuitive for some people.

Fix

Split the Lend command into Lend and Borrow commands to section out the direction of money flow intuitively, meaning that Lend would be money out (they owe you) whereas Borrow would be money in (you owe them).

soc-pe-bot commented 4 months ago

Team's Response

Very valid point, it is something that we actually considered, but since the MONEY_OWED parameter in add and edit command allow negative numbers to represent you owing money, and we did specify that in the user guide, using negative numbers in this command to represent “borrow” instead doesn’t seem too much of a stretch for users who are already familiar with how our application works.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]