search

Jaspertzx / NUS-CS2103T-pe

0 stars 0 forks source link

Terminal printout of application named is not updated to InternHub #7

Open Jaspertzx opened 7 months ago

Jaspertzx commented 7 months ago

Details: The application print out should be updated to it's name, such that the User (upon looking at the terminal) understand which program he/she/they/them is running, otherwise they may mistake themselves as running AB3 instead in the log files.

Steps to reproduce:

  1. Run the application on terminal
  2. View the terminal print out

Expected: Initialising InternHub Stopping InternHub

Actual: Initialising Address Book Stopping InternHub

Screenshots: image.png

image.png

Rationale: This is important especially when looking at the log files of a PC during recovery or forensics to look at which application or application information could be causing the issue.

nus-se-script commented 7 months ago

Team's Response

Dear tester,

Thank you for your feedback.

While we agree that the naming is different, we decided to follow the 'AddressBook' convention as in our code so as to have more accurate log results. We aim to change these naming conventions in the future iterations

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]