Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Exit command message should say closing Nursing Address Book instead of Address Book #5

Open
baskargopinath opened this issue Apr 19, 2024 · 1 comment
Open

Exit command message should say closing Nursing Address Book instead of Address Book #5

baskargopinath opened this issue Apr 19, 2024 · 1 comment
Labels
severity.VeryLow status.Done type.DocumentationBug

Comments

@baskargopinath
Copy link
Owner

baskargopinath commented Apr 19, 2024 •
edited
Loading

Screenshot 2024-04-19 at 4.50.40 PM.png

Upon exiting, the exit command message should say Stopping Nursing Address Book instead of Address Book
@soc-se-bot
Copy link

soc-se-bot commented Apr 22, 2024 •
edited by baskargopinath
Loading

Team's Response

This is not a user facing issue. Rather, it is a log for developers. Hence, it does not hinder the user in any way.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: image.png
nus-cs2103-AY2324S2/forum#752

as mentioned by prof, showing the old app name qualifies as an error as the log file is user facing.

Also, I feel that this is a functionality bug in the log files rather than a documentation bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
severity.VeryLow status.Done type.DocumentationBug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@baskargopinath @soc-se-bot