Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update to the new API #61

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update to the new API #61

wants to merge 3 commits into from

Commits on Dec 18, 2018

  1. Update to the new API 

    Here is the link to the new documentation:

https://docs.microsoft.com/en-us/linkedin/consumer/integrations/self-serve/sign-in-with-linkedin?context=linkedin/consumer/context

The api request now has to be:
`GET https://api.linkedin.com/v2/me`

It does not accept parameters any longer and the only scope that are valid are nor r_liteprofile and r_emailaddress which also requires its own api call
    @G-T-P
    G-T-P authored Dec 18, 2018
    Configuration menu
    Copy the full SHA
    6905b94 View commit details
    Browse the repository at this point in the history

  2. Missed some brackets

    @G-T-P
    G-T-P authored Dec 18, 2018
    Configuration menu
    Copy the full SHA
    1355d05 View commit details
    Browse the repository at this point in the history

  3. Went a bit fast in the first commit 

    Tested it and it works.
The details of waht is returned (which could be greatly improved upon) is documented here:
https://developer.linkedin.com/docs/ref/v2/media-migration
    @G-T-P
    G-T-P authored Dec 18, 2018
    Configuration menu
    Copy the full SHA
    6c2da37 View commit details
    Browse the repository at this point in the history