Feature #20214
closedFeature #19592: Groups Page Re-Design
Move group "Manage > Social Media Accounts" settings to group "Manage > Details" page
0%
Description
As per the group redesign mockups, the group "Manage > Social Media Accounts" page is going to merge into the group "Manage > Details" page.
The settings from the "Manage > Social Media Accounts" page belong to the `bp-groups-social-media` plugin created by Dan awhile ago. The mockups call for a simpler interface compared to the current Social Media Accounts admin UI.
Boone, I'm tentatively assigning this to you, but feel free to assign it back to me if you have other things to work on.
Files
Updated by Boone Gorges 8 months ago
- Status changed from New to Testing Required
Thanks for the ping on this, Ray. A first implementation is in https://github.com/cuny-academic-commons/cac/commit/e44fa1b7376fc2eacde43d37c09d222050109cd7 and is ready for testing on cdev.
I decided to eliminate the bp-groups-social-media plugin altogether. (it's still in the codebase for reference until this ticket is resolved.) The new plugin supports the old data format, which allowed you to enter URLs or usernames, depending on the service. But there's some JS in the new editing interface that encourages the use of full URLs. So if you enter 'boone' into the Twitter field, on blur it'll validate and swap out with https://twitter.com/boone.
Updated by Colin McDonald 7 months ago
At the risk of getting too into the weeds on URL validation, which I'm sure is complex, I noticed a couple of things while testing these settings for the new Group design:
- Is the X (Twitter) field set to recognize twitter.com links but not x.com links? I can't get the latter to register.
- If entering a full link, rather than the username only as Boone mentions above, does it only register if you add the https prefix? Should it recognize if I write x.com/boone, for example? Or is that even possible?
Updated by Boone Gorges 7 months ago
My initial implementation wasn't set up to recognize x.com or URLs without the https:// protocol. I've made a change in https://github.com/cuny-academic-commons/cac/commit/2b67696d50da31c7efa535e7799e7da78a702f5a that should accommodate both.
Updated by Colin McDonald 7 months ago
Thanks, Boone! Both are working for me now.
Updated by Boone Gorges 6 months ago
- Status changed from Testing Required to Resolved