seadragon2000341 / pe

0 stars 0 forks source link

Unable to add valid entry #4

Open seadragon2000341 opened 1 year ago

seadragon2000341 commented 1 year ago

add t/Pears d/Yummy i/Pear, one that is red! 1, piece, 0.50 s/Juice the orange, and then a pear!

image.png

Its a valid entry, since "Pear, one that is red!" Is a sentence. I rate this severity.medium as it this is highly probably amongst users who use commas in their sentences

soc-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]

Add Command: Instruction does not allow for dashes

I believe that titles should be allowed to accept hyphen-dashes, espcially since there are words that exist in the dictionary that have dashes.

Not frequent to have such words but may still confuse the user especially since the error message claims that a word/sentence was not given for the title when orange-juice is considered a word

image.png


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

Their Response to the 'Original' Bug

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

It is a feature flaw as we only take in alphanumeric and space when parsing user inputs for descriptions/titles/etc.

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: The other issue is phrased as a feature flaw ("titles should be allowed to accept hyphen-dashes"), but my issue is phrased as a functionality bug (user guide say sentences can be accepted, but actual behaviour is that sentences were not accepted). Due to the different type of bugs, these are not duplicates. See the next section for more details.


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** It is a functionality bug as it is "A functionality does not work as specified/expected" - based on the rubrics Expected: Able to add sentences to the ingredient field as specified in the user guide Actual: Unable to add sentence ("Pear, one that is red!") - > behaviour deviates from what is prescribed in the user guide
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** severity.Medium : A flaw that causes occasional inconvenience to some users but they can continue to use the product. Users who want to add sentences like the one above (Pear, one that is red!) in the ingredients field are unable to do so. It causes occasional inconvenience to such users (need to rephrase/change the sentence), but they can continue to use the product. This fits into the description of severity.medium.