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

[ISSUE 48459] Source-Instagram: Remove Unsupported Fields in media_insights and user_insights #51011

Conversation

nickzombolas
Copy link

What

Solves #48459
Starting January 8, 2025, the following fields are no longer supported in the following streams. Here are the Facebook docs as a reference: https://developers.facebook.com/docs/graph-api/changelog/version21.0

user_insights: video_views
user_insights: email_contacts, get_direction_clicks, profile_views, text_message_clicks, website_clicks, phone_call_clicks

Including these fields results in a bad request and causes these streams to fail syncs.

How

Unsupported fields have been removed from requests. Schemas and ERDs have been updated

Review guide

Ensure All unsupported fields are removed from requests

User Impact

These metrics are no longer available, and will no longer be fetched in records. If users depend on these fields they will need to migrate away from them as Facebook will no longer provide them

Can this PR be safely reverted and rolled back?

  • YES 💚
  • [ x] NO ❌

Because rolling back this PR will produce bad requests and cause syncs to fail.

Copy link

vercel bot commented Jan 9, 2025

@nickzombolas 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 Jan 9, 2025

CLA assistant check
All committers have signed the CLA.

@octavia-squidington-iii octavia-squidington-iii added the area/connectors Connector related issues label Jan 9, 2025
@nickzombolas nickzombolas changed the title [ISSUE 48459] Remove Unsupported Instagram Fields [ISSUE 48459] Source-Instagram: Remove Unsupported Fields in media_insights and user_insights Jan 9, 2025
@marcosmarxm
Copy link
Member

Thanks for the quick update in this deprecation @nickzombolas

@marcosmarxm marcosmarxm requested review from maxi297 and a team January 9, 2025 20:49
@marcosmarxm
Copy link
Member

I requested the team to review your contribution @nickzombolas

Copy link
Member

@marcosmarxm marcosmarxm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a entry to the connector documentation (docs/integration...) and update the metadata.yaml and pyproject.toml connector version inside the connector folder.

@marcosmarxm
Copy link
Member

Thanks @nickzombolas the team is already working #51018 to get this fixed.

@marcosmarxm marcosmarxm closed this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants