EdmundTangg / ped

0 stars 0 forks source link

Add-Edit product design #7

Open EdmundTangg opened 5 months ago

EdmundTangg commented 5 months ago

As a user, I can choose to add in the date of my transaction (optional).

However as a user, i cannot edit the date of my transaction, which feels weird because you gave me the freedom to enter something but not edit it.

My suggestion would be to standardise, either

  1. remove the date field in the add transaction feature
  2. Allow the user to edit the date field in the edit transaction feature
1simjustin commented 5 months ago

Hi, thanks for your review! This functionality seems to be working as intended on our end, could you provide the steps to replicate this issue? For reference, this is the behaviour we received when testing this functionality (excluding log messages). Do let us know if you've had a different experience yourself.

Enter command: add member Alice
Enter command: add member Bob
Enter command: add transaction Alice t/23-04-2024 1200 p/Bob a/5
Enter command: list transactions
1.
Lender: Alice
Transaction time: 23 Apr 2024 12:00PM
Borrower 1: Bob Owed amount: 5.00

Enter command: edit transaction 1 Bob t/24-05-2025 2300 p/Alice a/10
Enter command: list transactions
1.
Lender: Bob
Transaction time: 24 May 2025 11:00PM
Borrower 1: Alice Owed amount: 10.00
EdmundTangg commented 5 months ago

Oh, as a user I thought that was not possible because the UG did not mention any traces of me being able to edit the time.

image

Perhaps your UG can be more clear to the users that the optional field is editable as well! This can be indicated under the format heading.

I tried on my end now and it works fine.

Apologies for any inconvenience caused.