g-tejas / pe

0 stars 0 forks source link

Valid names containing "/" not allowed #2

Closed g-tejas closed 4 months ago

g-tejas commented 4 months ago

Description

When adding a person with the name "Muthu s/o Samy" which is a valid name which many people have the character "/", results in the following error. However, it is enforced to be alphanumeric only.

Screenshots

CleanShot 2024-04-19 at 16.20.07@2x.png

Steps to reproduce

  1. Run command: add n/Muthu s/o Samy p/98765432 e/johnd@example.com m/CS2103 f/Computing v/Johnstreet, block 123, #01-01

Expected vs Actual Behaviour

The app should allow such names, however, there's an error thrown saying "Name can only contain ..."

Suggestion

Perhaps modify the prefix multimap code to allow this? Or can change the prefix to "=", since that isn't used in valid names.