Work history, education and skills #28

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@sorin

sorin commented Jun 19, 2012

Hello,

First of all, congrats on a very useful and well written library. I'd like to contribute by developing a couple of new features we need:

Hybrid_Provider_Model->getUserWorkHistory()
Hybrid_Provider_Model->getUserEducation()
Hybrid_Provider_Model->getUserSkills()

These would work in a similar way to getUserProfile and getUserContacts. The initial implementation will be for Facebook/Linkedin

See the attached pull request.

Sorin

@mastermunj

This comment has been minimized.

Show comment
Hide comment
@mastermunj

mastermunj Oct 29, 2012

@hybridauth, This seems a nice & helpful change. Do you think this could be merged in?

@hybridauth, This seems a nice & helpful change. Do you think this could be merged in?

@StorytellerCZ

This comment has been minimized.

Show comment
Hide comment
@StorytellerCZ

StorytellerCZ Mar 4, 2014

Contributor

Might be considered for 3.x branch, but I'm not sure how many people would utilize it.

Contributor

StorytellerCZ commented Mar 4, 2014

Might be considered for 3.x branch, but I'm not sure how many people would utilize it.

@sorin

This comment has been minimized.

Show comment
Hide comment
@sorin

sorin Mar 4, 2014

It's been 2 years since the original PR, I no longer use hybridauth (or PHP for that matter), so I dont know if that code even works anymore. If it's useful to anybody, feel free to use it, otherwise delete.

sorin commented Mar 4, 2014

It's been 2 years since the original PR, I no longer use hybridauth (or PHP for that matter), so I dont know if that code even works anymore. If it's useful to anybody, feel free to use it, otherwise delete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment