search

Jellybeano / pe

0 stars 0 forks source link

Notes that are no longer relevant #15

Open Jellybeano opened 3 years ago

Jellybeano commented 3 years ago

The note as shown below still mentions the AddressBook that the current application was morphed from

Would be nice if it at least updated to say Groups instead of Tags to stay consistent with the rest of the diagrams

image.png

nus-pe-bot commented 3 years ago

Team's Response

Thanks for the comment.

We do have an AddressBook in our code, which we do derive from the original AB3 and so it is intended to be in the diagram.

In this specific section, we consider an alternative implementation (i.e. not the one we have) so we deliberately use terms that are different from our actual implementation to make this clear.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Fair enough, but having different terms from the actual implementation would probably confuse readers further.

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]