Similar to #671622: Remove duplication between name and fullname but for the Storm org module.
I propose to remove both the prefix and fullname fields.
Any thoughts?
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | storm--685140-3.patch | 4.17 KB | juliangb |
Similar to #671622: Remove duplication between name and fullname but for the Storm org module.
I propose to remove both the prefix and fullname fields.
Any thoughts?
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | storm--685140-3.patch | 4.17 KB | juliangb |
Comments
Comment #1
JGonzalez commentedI agree with proposal on this issue more than on person.
I never knew what to put in fullname and prefix for Storm Org. fullname of organization? fullname of main contact?
Does anyone use these fields for different purposes?
Comment #2
tchurch commentedI agree. I've been helping a customer implement a Storm installation and they keep asking about this.
I would have thought that Prefix would be more useful under Person (i.e. Mr, Mrs, Miss or whatever, controlled through attributes).
Comment #3
juliangb commentedA patch for testing.
Does not remove data from db.
I'm undecided over whether to do this with this change immediately or give users a grace period where the data is hidden but present before putting in an update function to remove it.
Comment #4
bigjim commented+1 to drop the fields.
On the grace period question, the question is how do you warn people? Do you make Full Name an option that can e turned off and on in the Organization settings?
Comment #5
juliangb commentedWarn people via release notes. The reason that we have done grace periods for things like this in the past (generally ~3months or a couple of releases later) is because some people won't read the release notes (unadvised anyway!)
It isn't a setting, this is meant to be a simplification by removing something that is unnecessary in 99% of cases.
Comment #6
juliangb commentedNeeds retesting against 2.x.
Comment #7
juliangb commented#3: storm--685140-3.patch queued for re-testing.
Comment #8
juliangb commentedCommitted to 2.x. Will not be backported to 1.x.
Created #1046930: Clean up prefix and fullname data to handle the data cleanup followup.
Comment #9
juliangb commented