search

Bisceto / pe

0 stars 0 forks source link

DG still references AddressBook in commons package #11

Open Bisceto opened 1 year ago

Bisceto commented 1 year ago

image.png

Code should have been refactored to fit LeTracker

nus-pe-script commented 1 year ago

Team's Response

Thanks for reporting this issue.

Currently, the codebase does still use the "seedu.addressbook" package. We did consider changing it to "seedu.letracker", however, as per our discussion on the forum with prof Damith, it is recommended that we keep it as it is in order not to cause an impact on the Reposense.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]