nus-pe-bot
commented
9 months ago Team's Response
This is not a bug related to the quality of documentation, but is closer to a feature flaw since the feature was designed like that for v1.4, so we are rejecting this for that reason.
That said, even if it wasn't submitted late: As a feature flaw (which we cannot accept since it was reported in Part II stage of the PE anyways), we agree with your statement that "This can be quite confusing for users, perhaps a better way to implement this is to provide separate prefixes for people that owe you and people that you owe", and we think this makes sense as an indictment to the ergonomics of using the find command with balances (rather than an actual inconsistency with money format in our docs). This is mentioned in the Planned Enhancements section of our DG (planned enhancement 9)
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
It is interesting to see that the money format of the app disallows negative signs when working with the 'pay' or 'owe' command. Yet, searching for contacts based on balance with the 'find bal/' command allows negative signs. This can be quite confusing for users, perhaps a better way to implement this is to provide separate prefixes for people that owe you and people that you owe?