`Parser` lifecycle seems to be incorrectly represented in sequence diagram

ltzehan / pe

0 stars 0 forks source link

Open ltzehan opened 1 year ago

ltzehan commented 1 year ago

In the "Adding Appointments" sequence diagram, some parts seem to be misrepresented:

Ui does not call Parser::parserAppointmentInput; this is done in the Sniff::run method
The Parser instance is deleted after returning from parseAppointmentInput; this does not seem to be the case as parseAppointmentInput is a static method call and should also be marked as such

nus-pe-script commented 1 year ago

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