Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[PE-D][Tester B] Gender entry for adding new resident is case sensitive #212

Closed
soc-pe-bot opened this issue Oct 28, 2022 · 1 comment · Fixed by #278
Closed

[PE-D][Tester B] Gender entry for adding new resident is case sensitive #212

soc-pe-bot opened this issue Oct 28, 2022 · 1 comment · Fixed by #278
Assignees
@nareus
Labels
severity.VeryLow A flaw that is purely cosmetic and does not affect usage type.Bug Something isn't working type.Chore Something that needs to be done, but not a story, bug, or an epic
Milestone
v1.4

Comments

@soc-pe-bot
Copy link

image.png

Not a serious issue, however the gender field should be case-insensitive to improve user experience instead of forcing users to capitalize the gender at every entry.

Labels: severity.VeryLow type.FeatureFlaw
original: Isaaclhy00/ped#1
@jq1836 jq1836 added type.Bug Something isn't working type.Chore Something that needs to be done, but not a story, bug, or an epic severity.Low A flaw that is unlikely to affect normal operations of the product beyondV1.4 and removed type.Bug Something isn't working type.Chore Something that needs to be done, but not a story, bug, or an epic severity.Low A flaw that is unlikely to affect normal operations of the product labels Oct 29, 2022
@jq1836
Copy link
Collaborator

jq1836 commented Nov 1, 2022

Case-insensitivity was intended as a behaviour of our commands to improve usability, however, we missed out the case-insensitivity update on Gender, House and MatricNumber fields.

@jq1836 jq1836 added type.Bug Something isn't working type.Chore Something that needs to be done, but not a story, bug, or an epic and removed beyondV1.4 labels Nov 1, 2022
@jq1836 jq1836 added the severity.VeryLow A flaw that is purely cosmetic and does not affect usage label Nov 1, 2022
@jq1836 jq1836 added this to the v1.4 milestone Nov 1, 2022
@nareus nareus mentioned this issue Nov 2, 2022
Merged
@jq1836 jq1836 linked a pull request Nov 3, 2022 that will close this issue
BUG FIX Command bugs #278
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nareus nareus

Labels
severity.VeryLow A flaw that is purely cosmetic and does not affect usage type.Bug Something isn't working type.Chore Something that needs to be done, but not a story, bug, or an epic
Projects
None yet
Milestone
v1.4
Development

Successfully merging a pull request may close this issue.

BUG FIX Command bugs AY2223S1-CS2103T-W12-3/tp
3 participants
@nareus @soc-pe-bot @jq1836