Feature #2788
closedCommons Profile: College and Title
0%
Description
Hi all,
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.
Best,
Sarah
Updated by Chris Stein over 10 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.
Updated by Boone Gorges over 10 years ago
- Assignee set to Chris Stein
- Target version set to 1.6
I suggest that, when viewing the Commons Profile of a member who has Positions filled in, do not display College and Title (unless it's your own profile).
Updated by Boone Gorges about 10 years ago
- Category name changed from Commons Profile to Public Portfolio
Updated by Chris Stein about 10 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
Updated by Boone Gorges about 10 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.
Updated by Boone Gorges about 10 years ago
- Assignee changed from Boone Gorges to Chris Stein
Updated by Chris Stein about 10 years ago
OK, looks good to me. For some reason I can't change status to resolved.