ianyong / pe

0 stars 0 forks source link

Developer Guide: Unclear description of main #16

Open ianyong opened 3 years ago

ianyong commented 3 years ago

image.png

Main has two classes called Main and MainApp.

This sentence is confusing as no context is given to the first instance of Main in the sentence. Nowhere is it stated that it is a component or a class, or whatever it is supposed to be.

nus-se-bot commented 3 years ago

Team's Response

  1. It is stated before the description, "Given below is a quick overview of each component." Thus, it is implied that the first instance of Main is a component.
  2. This issue is inherited from AB3, and we are told that this can be rejected. You can reject bugs that you inherited from AB3. (https://nus-cs2103-ay2021s1.github.io/website/admin/tp-pe.html#pe-phase-2-developer-response-2)

Screenshot 2020-11-17 at 4.05.18 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]