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-ads] Lead forms & Lead form responses new streams #52047

Merged

Conversation

kev-datams
Copy link
Contributor

@kev-datams kev-datams commented Jan 21, 2025

What

This Pull Request aims in proposing a first version of the LinkedIn Leads streams based on official documentation.

It follows this PR adapted for low-code and merged in LinkedIn Ads connector.

How

This PR adds 2 base streams in existing LinkedIn Ads connector:

  • lead forms
  • lead form responses

The required permission on LinkedIn is r_marketing_leadgen_automation according to this doc bloc.

Review guide

  1. manifest.yaml
  2. Documentation (notably the Limits & considerations section)

User Impact

User will have access to 2 streams of a new connector.

As explained in the Limits & considerations section of the Documentation and due to low-code capabilities limitations:

  • some query params have been hard-coded in the path
  • Incremental sync is not supported because of specific query param expected by LinkedIn API (eg: submittedAtTimeRange=(start:1711407600000,end:1711494000000))

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌ (as soon as it is merged, customers could potentially use it in production)

Copy link

vercel bot commented Jan 21, 2025

@kev-datams is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues area/documentation Improvements or additions to documentation community connectors/source/linkedin-ads labels Jan 21, 2025
@kev-datams kev-datams changed the title [Source LinkedIn Ads] - New streams Lead forms & Lead form responses source-linkedin-ads - New streams Lead forms & Lead form responses Jan 21, 2025
@kev-datams
Copy link
Contributor Author

hello @natikgadzhi @marcosmarxm, FYI following this previous PR 🙏

…-leads-streams

# Conflicts:
#	airbyte-integrations/connectors/source-linkedin-ads/metadata.yaml
#	airbyte-integrations/connectors/source-linkedin-ads/pyproject.toml
#	docs/integrations/sources/linkedin-ads.md
Copy link

vercel bot commented Feb 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 2:59pm

@natikgadzhi
Copy link
Contributor

I think the CATs are failing because our sandbox does not have the data in the new streams. Can you edit the acceptance-test-config to ignore the new streams please?

auto-merge was automatically disabled February 6, 2025 20:55

Head branch was pushed to by a user without write access

@kev-datams kev-datams temporarily deployed to community-ci-auto February 6, 2025 20:55 — with GitHub Actions Inactive
@kev-datams
Copy link
Contributor Author

I think the CATs are failing because our sandbox does not have the data in the new streams. Can you edit the acceptance-test-config to ignore the new streams please?

@natikgadzhi done ✅

@natikgadzhi
Copy link
Contributor

@DanyloGL can you please test this one?

@marcosmarxm marcosmarxm merged commit 7df5607 into airbytehq:master Feb 7, 2025
27 checks passed
@kev-datams kev-datams deleted the source-linkedin-ads-leads-streams branch February 7, 2025 15:40
@natikgadzhi
Copy link
Contributor

Really nice work @marcosmarxm and @kev-datams. @kev-datams thank you for being so patient with us and so generous with your contributions <3

@kev-datams
Copy link
Contributor Author

Really nice work @marcosmarxm and @kev-datams. @kev-datams thank you for being so patient with us and so generous with your contributions <3

Thank you for reviewing, this is clearly a win-win 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation community connectors/source/linkedin-ads
Projects
Development

Successfully merging this pull request may close these issues.

4 participants