yaojiax / pe

0 stars 0 forks source link

аdd does not work with Cyrillic а, lack of specific error message to explain that its not possible #9

Open yaojiax opened 1 year ago

yaojiax commented 1 year ago

image.png

аdd command doesn't work when the a is replaced by Cyrillic a. As an international company my employees using Cyrillic а may not use this app as intended

To replicate: аdd n/test p/98765432 d/Marketing pr/1000 15 e/johnd@example.com a/John street, block 123, #01-01

nus-se-bot commented 1 year ago

Team's Response

same issue due to non-standard characters

The 'Original' Bug

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

edit does not work with Cyrillic e

image.png

Similarly, edit does not work if the e is a Cyrillic e. As an international company, this may confuse my employees using a mix of Cyrillic and English characters.

To replicate: еdit 1 p/91234567 e/johndoe@example.com


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

Their Response to the 'Original' Bug

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

Currently our app is focusing on start up companies in Singapore, majority of the companies also use normal English characters, so it is definitely still a feature to consider when expanding globally. The companies should also have a standardisation of using a single language like English, in order to ensure consistent workflow, so the mix of alphabets is less likely to be used.

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.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]