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.

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

source linkedin-pages🐛: Fix API version #55816

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

Conversation

OwenPolybox
Copy link

What

The LinkedIn API version currently being used by the LinkedIn-Pages connector has been deprecated, causing all existing LinkedIn Pages sources to fail #55794.

Following the proposed solution in #55794:

Following the documentation it is possible to use the 20250301 version without any schema or connector changes more than changing the version in the yaml file

The API version for linkedin-pages has been updated from 202403 to 20250301

How

Within the manifest.yaml for LinkedIn Pages, finding and replacing 202403 with 20250301.

Review guide

User Impact

The values/schema returned from the API will be the same so there should be no impact.

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Copy link

vercel bot commented Mar 18, 2025

@OwenPolybox is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Mar 18, 2025

CLA assistant check
All committers have signed the CLA.

@OwenPolybox OwenPolybox changed the title source linkedin-pages: Fix API version source linkedin-pages🐛: Fix API version Mar 18, 2025
@natikgadzhi
Copy link
Contributor

Dammit, we missed this one. Thank you for this! Can you also bump the version and add a changelog in the docs? It's in docs/integrations/sources/linkedin-pages.md.

@OwenPolybox
Copy link
Author

Dammit, we missed this one. Thank you for this! Can you also bump the version and add a changelog in the docs? It's in docs/integrations/sources/linkedin-pages.md.

@natikgadzhi sorry for the oversight! I've commited the change to the changelog now

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

Successfully merging this pull request may close these issues.

4 participants