Project

General

Profile

Feature #7292

LinkedIn link on member page

Added by Raffi Khatchadourian almost 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
-
Target version:
Start date:
2017-01-01
Due date:
% Done:

0%

Estimated time:

Description

Could there be a form to add a LinkedIn profile URL at the member page (e.g., https://commons.gc.cuny.edu/members/khatchad)? There is already one for twitter and GitHub. Thanks!

bp-social-media-profiles.diff (1.49 KB) bp-social-media-profiles.diff Boone Gorges, 2017-01-05 11:38 AM

History

#1 Updated by Raffi Khatchadourian almost 3 years ago

Interestingly, there is a field for LinkedIn on https://commons.gc.cuny.edu/members/khatchad/profile/edit/group/1/. But, despite populating that field, I don't see a LinkedIn line on the profile.

#2 Updated by Boone Gorges almost 3 years ago

Thanks for the report, Raffi.

There was a bug in the way that the Commons's profile fields were being mapped onto the social media sites, such that the lookup was failing when the name of the profile field (such as "LinkedIn Profile Link") wasn't exactly the same as the social network's name ("LinkedIn").

Dan, I think you were the last to touch this. I've attached the patch that I've applied as a hotfix to the production site. It seems to correct the logic for me, but I would like a second look from you.

#3 Updated by Boone Gorges almost 3 years ago

  • Target version changed from 1.10.7 to 1.10.8

#4 Updated by Raffi Khatchadourian almost 3 years ago

I see it.

#5 Updated by Boone Gorges almost 3 years ago

Thanks, Raffi.

Dan, I've still got the hotfix in place. When you get a chance, please have a look to see if my change makes sense.

#6 Updated by Boone Gorges almost 3 years ago

  • Target version changed from 1.10.8 to 1.10.9

#7 Updated by Daniel Jones almost 3 years ago

Hey Boone and Raffi - sorry about the long delay here. The change looks right to me. Should I push it to 1.10.x or is it better for you to handle that end, Boone?

#8 Updated by Boone Gorges almost 3 years ago

Cool - thanks, Dan. If you could implement and push, that'd be great. Thanks!

#9 Updated by Boone Gorges almost 3 years ago

  • Target version changed from 1.10.9 to 1.10.10

#10 Updated by Boone Gorges almost 3 years ago

  • Target version changed from 1.10.10 to 1.10.11

#11 Updated by Boone Gorges over 2 years ago

  • Target version changed from 1.10.11 to 1.10.12

#12 Updated by Boone Gorges over 2 years ago

  • Target version changed from 1.10.12 to 1.10.13

#13 Updated by Boone Gorges over 2 years ago

  • Target version changed from 1.10.13 to 1.10.14

#14 Updated by Daniel Jones over 2 years ago

#15 Updated by Boone Gorges over 2 years ago

  • Status changed from Assigned to Resolved

Thanks!

Also available in: Atom PDF