You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I do not think there should be _ used at all, also the object name should not be the description of the note itself, it is a field of the Note object instead.
The text was updated successfully, but these errors were encountered:
Thank you for pointing it out! The bug is purely cosmetic and does not affect the understanding of how the "noteCompany" command work and is unlikely to confuse the reader, hence we have decided to change it to Severity.VeryLow.
Items for the Tester to Verify
❓ Issue severity
Team chose [severity.VeryLow]
Originally [severity.Low]
I disagree
Reason for disagreement: For this case, I am trying to point out that the object name shouldn't be the description of the note itself. This purely is a incorrect storing of the note description, rather should be shown as an attribute instead. This violates the coding style as well if I am not wrong.
For example:
usually it will be
Note newNote = new Note("Looking for aspiring frontend developers");
rather than:
Note Looking for aspiring frontend developers= new Note();
This shows the misunderstanding of the use off object diagram. That is why I flagged it out as severity.Low
I do not think there should be _ used at all, also the object name should not be the description of the note itself, it is a field of the Note object instead.
The text was updated successfully, but these errors were encountered: