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.

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

Startup message says "Address Book" instead of "NursingAddressBook" #2

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

Comments

@baskargopinath
Copy link
Owner

baskargopinath commented Apr 19, 2024

image.png

As mentioned by prof, maybe the startup message should match your product name NursingAddressBook instead of Address Book as it may confuse the user that they are opening a different application.

@soc-se-bot
Copy link

soc-se-bot commented Apr 22, 2024

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. Furthermore, both names specify that it is an address book, thus it is unlikely to confuse the user. Perhaps you can link where the prof may have mentioned this.

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.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants