gnoznaug / pe Goto Github PK

Upon clicking the word clear, it does not bring me to the clear command unlike the link for listMembers.

It would make sense for the owner of a small business to want to keep track of which clients bring the most profits. Either by number of transactions or the total amount of money of the transactions.

Imagine if a client just paid for all their 10 transactions. It would be troublesome to type out pay 1, pay 2, etc.

It would make more sense for a command to set all current transactions on screen to be paid. This could be used in conjunction with Find Client by Keyword command to search for the client and set all their transactions to be paid.

While I understand that there is a undo feature to undo the clearing of data, I believe it is a good idea to either have a word of caution below the command description, or have a hyperlink to the undo command, telling the reader that they do not have to worry and can just use the undo command.

This line being pointed to by the red arrow should be solid rather than dotted.

It does not make sense that two different clients can share the same phone number. If two clients having the same email is not allowed, then it would make sense that two clients having the same phone number is also not allowed.

The link for MainWindow.fxml links to the MainWindow.fxml for AB-3. This might confuse the reader as previously the DG mentions the TransactionListPanel, but this does not exist in AB-3.

This link is in the UI section of the DG.

The 2 Persons underlined should be Person to follow formatting.

When the client is deleted, all of its transactions are also deleted.

It should be When the client is deleted, all of their transactions are also deleted.

Long tags will stretch out far beyond the screen and is unable to be seen.

The underlined text should be 1b1 instead.

The user should not have to manually set a "free" transaction to be paid.

addTransaction 1 a/0 td/2020-11-11

The AMOUNT specified must be between 0 and 9999999.99 inclusive. The number will be rounded to two decimal places.

This will make the reader unsure whether the amount will be rounded up or down or to the closest 2dp.

Might be a good idea to include that.

This way the user can check when which transactions are due soon.

The link for MainWindow.java links to the MainWindow.java for AB-3. This might confuse the reader as previously the DG mentions the TransactionListPanel, but this does not exist in AB-3.

This link is in the UI section of the DG.

As there are no restrictions on remark, the user should be able to render emojis or even funny looking characters such as ヽ༼ຈ ل͜ຈ༽ﾉ ヽ༼ຈ ﾉ. However, upon adding a remark to a client, it is not rendered properly.

remark 1 r/ヽ༼ຈ ل͜ຈ༽ﾉ ヽ༼ຈ ل͜ຈ༽ﾉ

The underlined text should be 1b1 instead.

This line being pointed to by the red arrow should be solid.