Jellybeano / pe

0 stars 0 forks source link

Adding of meetings that have already passed #3

Open Jellybeano opened 3 years ago

Jellybeano commented 3 years ago

Users are allowed to add meetings that have already started and ended, perhaps this might lead to problems as usually no user would do that and likely doing so is because of a typo?

image.png

However, as shown above, the application allows such a request to go through

nus-pe-bot commented 3 years ago

Team's Response

This is intended.

The 'Original' Bug

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

Can add meeting that have already passed

I am able to add a past date which shouldnt be the case

image.png


[original: nus-cs2103-AY2021S2/pe-interim#2735] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

This is because we want users to be able to keep track of passed meetings and schedules . Setting the meetings to be unable to add pass meetings will add unnecessary layer of complexity and does not improve flexibility for the user. A user can just choose not to add meetings that have already been passed, assuming the user knows today's date. Furthermore he can easily check the date's correctness by searching for the meeting card and viewing the date.

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: Arguably, not having any new past meetings in the application would remove a level of complexity for the user instead.


:question: Issue type

Team chose [type.FunctionalityBug] Originally [type.FeatureFlaw]

Reason for disagreement: It is not a bug as the team did not state anywhere that the command cannot take in dates that already have passed, however it is a feature flaw as the adding of past dates is not helpful to the meeting feature.