Commons Profile: College and Title
Now that users can no longer edit their college and title fields on their Commons profile, what should be done with them? Should they be removed, hidden, etc?
I ask because a member recently reported a spelling error in their title and they no have no way of updating it. Boone suggested that I create this ticket for the team to discuss the future of these fields so I'm adding the members of the Commons team as watchers to this ticket.
#1 Updated by Chris Stein about 8 years ago
Thanks for starting this Sarah. Part of the problem before the 1.5.2 update is that there was nothing telling people to use the Positions widget instead. Now that it's there, hopefully that will be clearer.
There is still the question of how long to keep showing the College and Title fields as we move forward. I haven't thought about it much but my first inclination is to say that at some point we can stop showing the fields and the message if the user has Positions and then think about whether we might at some point remove them entirely for everyone.
#6 Updated by Chris Stein over 7 years ago
- Status changed from New to Assigned
- Assignee changed from Chris Stein to Boone Gorges
Agree with Boone's suggestion. I would go a bit farther and suggest that:
- Don't show fields ever when viewing other's profiles
- If the user has Positions filled then don't show the fields or the message to the user either
- Only If the user both doesn't have Positions and has something in College and Title, then you can show message and fields
I'm pretty sure that new users don't ever see those fields so it's not a problem for them. Correct me if I'm wrong.
moving to Boone for implementation
#7 Updated by Boone Gorges over 7 years ago
- Status changed from Assigned to Testing Required
In https://github.com/castiron/cac/commit/3791cf48b69459f3df87408110ec606c99c44f80 I've changed the logic so that these legacy fields + message are only available if ( it's your profile && you have no Positions widget && you have filled in either the Titlle or College widget ). Ready to test on cdev.