search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

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

Open nus-pe-bot opened 5 months ago

nus-pe-bot commented 5 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.

[original: nus-cs2103-AY2324S2/pe-interim#3148] [original labels: type.FunctionalityBug severity.VeryLow]

eunrcn commented 5 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

Duplicate status (if any):

--