nus-cs2103-AY1819S2 / forum

CS2103/T discussion forum
6 stars 1 forks source link

Review the bug reports received by your team #86

Open ccristina opened 5 years ago

ccristina commented 5 years ago

Dear Students,

Before processing the bugs, carefully read the instructions here: https://nus-cs2103-ay1819s2.github.io/cs2103-website/admin/project-deliverables.html#processing-pe-bug-reports%3A-2.

Few things to quickly point out:

Please note that you are allowed to change the severity and type of the bug. However, you need to provide an explanatory comment to justify your choice.

Best regards, Cristina.

lingyanhao commented 5 years ago

I am unable to access the repo and got a 404. Is anyone else facing the same issue?

image

ccristina commented 5 years ago

You should see it now! Check again!

Jackimaru96 commented 5 years ago

Will bug reports that are 'insincere/random' be penalised and can the team reviewing the bug reports report if we deem them to be insincere?

image

okkhoy commented 5 years ago

@Jackimaru96 What do you mean?

Jackimaru96 commented 5 years ago

@Jackimaru96 What do you mean?

Thank you for your reply Prof @okkhoy I meant for bug reports that didn't put in effort to show how it can be reproduced/ for example purposely used parameters that are already stated to not be used in the UG like for example input values like 05.50 instead of 5.50

okkhoy commented 5 years ago

Well, without any other context, this particular example is a bug. You should handle such inputs in your code, irrespective of documenting it.

This can also be reproduced, just use 05:50 instead of 5:50 as input.

An example of Cannot be reproduced could be something like just one screenshot without any explanation.

Jackimaru96 commented 5 years ago

Well, without any other context, this particular example is a bug. You should handle such inputs in your code, irrespective of documenting it.

This can also be reproduced, just use 05:50 instead of 5:50 as input.

An example of Cannot be reproduced could be something like just one screenshot without any explanation.

@okkhoy Thank you for the clarification