pkpaing / pe

0 stars 0 forks source link

Editing name of module #9

Open pkpaing opened 1 year ago

pkpaing commented 1 year ago

Screenshot 2023-04-14 at 3.08.40 PM.png How to replicate: edit 8 Bt3103

The application does not detect when I am simply editing the casing of the module. In my opinion it should reject the edit command in this case as the module is the same and it is just the casing that is changed.

nus-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Feature Flaw - Cannot differentiate between uppercase and lowercase in name

Steps to recreate:

  1. Normal command: `add n/CS2103T t/Lecture

image.png

  1. Erroneous command: add n/cs2103t t/Lecture

image.png

Both should be treated as the same but are considered different:

image.png


[original: nus-cs2103-AY2223S2/pe-interim#4386] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Thanks for the suggestion. We agree that this check could have been implemented. However, we think that this is likely to only cause a minor inconvenience and would likely only happen in rare scenarios. Hence, we have appointed a severity of low.

Thanks again for the feedback.

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 severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]