bbbbcccd / pe

0 stars 0 forks source link

Invalid command in proposed undo / redo feature #8

Open bbbbcccd opened 1 week ago

bbbbcccd commented 1 week ago

A person has name, job, phone number ... fields. However, in this diagram, the add command is shown to only include the name field, which can lead to misunderstandings for developers working on this feature in the future.

Screenshot 2024-11-15 at 5.23.28 PM.png

Justification: Severity.Low as this is not a cosmetic issue but rather an inaccuracy in command format in the diagram, which can lead to mistakes, especially for developers who just starting to contribute.

nus-se-script commented 1 week ago

Your response not required for this bug as the team has accepted the bug as it is.