yongchuann / pe

0 stars 0 forks source link

Log file naming is incorrect #10

Open yongchuann opened 3 years ago

yongchuann commented 3 years ago

image.png Expected: logfile to be renamed to the app name

As this is not addressbook, the log file of the app should be renamed accordingly else if the user needs to look for the log file, they might be confused with where the log file is.

nus-pe-bot commented 3 years ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

log file is addressbook.log

actual: log file is addressbook.log, which does not match the software name

expected: log file should match software name


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Renaming is optional and the team decided to leave the name of the log file as it is.

image.png

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Their argument shows that its optional to rename packages, but the log file is not a package. Don't agree with their reason as the log file will be seen by the user unlike the packages in the code