-
Notifications
You must be signed in to change notification settings - Fork 12
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.
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
Standardize Names and Encourage the Use of the Organization's Naming Standard #201
Comments
Discussed with @dspCarlson, the main goal here is to avoid the mess of all different ways of writing names by either suggesting or enforcing a naming convention.
|
A good starting read is https://www.kalzumeus.com/2010/06/17/falsehoods-programmers-believe-about-names/ |
Some examples of different names around the world are given in https://www.w3.org/International/questions/qa-personal-names , followed by some advice on what (not) to do. |
Basically, what W3 says, the only sensible way to store names is to use two fields: |
Preferred approach updated to the following:
|
new person Changes to be committed: modified: src/components/FormField.js new file: src/components/NameInput.css modified: src/pages/people/Form.js
Test failed: Verify the focus/cursor moves from the last name field to the first name field when a comma is entered and the focus is in the last name field. Two distinct bugs:
Condition:
Action:
Actual Result:
|
Test failed: The last name field displays a hint when the field is blank Bug: The first letter in the last name field cannot be removed Condtion:
Action:
Actual Result: |
Thanks @dspCarlson great finds |
Test failed: When saving a principal, the principal profile cannot be saved when the principal only has one name Many principals only have one name. The principal's signle name should be saved in the last name field and the first name field should be blank. |
@dspCarlson thanks for the feedback! The bug was introduced by #194 Have tested both the original issue 201 and 194 and the bug should be fixed in the next pull request :) |
GH-201-fix - Standardize Names
Standardize the way names are stored by validating names when saving:
Reference item 101
The text was updated successfully, but these errors were encountered: