tanyizhe / pe

0 stars 0 forks source link

Unintuitive application design choice #4

Open tanyizhe opened 1 year ago

tanyizhe commented 1 year ago

image.png

Cards are identified by an ID, but the ID is too long and random for the user. This ID provides no information of value to a user at all. Perhaps instead of displaying the ID here, display "Tag 5 has been removed from card 1" instead.

nus-se-script commented 1 year ago

Team's Response

Users are given two choices to identify with cards, i.e through -c CARD_UUID or through -i CARD_INDEX.

The reason why we do not display "Tag 5 has been removed from card 1" is because it presents some ambiguity in the sense that a card with cardIndex = 1 now might not be the same card after some deletion or addition of new cards, so we choose to display card UUID in the message because the UUID of a card will never change but the CARD_INDEX might change (depending on the index in card list)

But even if what the author said is true, this can be considered a suggestion for next version and not a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]