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 Amazon Ads: Added targeting grouping into SponsoredBrandsV3ReportStream. #48756

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

Conversation

mihaly-dombi
Copy link

@mihaly-dombi mihaly-dombi commented Dec 2, 2024

What

Sponsored brands can be fetched through targeting report type grouping.

How

Extending METRICS_MAP_V3 with targeting in SponsoredBrandsV3ReportStream.
Extending sponsored_brands_v3_report_stream.json with the new metrics.

Review guide

User Impact

They should get records with targeting record type.

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Copy link

vercel bot commented Dec 2, 2024

@mihaly-dombi 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 the area/connectors Connector related issues label Dec 2, 2024
@mihaly-dombi mihaly-dombi force-pushed the mihaly/extend-metrics-for-sponsored-brands branch from 18dac8c to 3a77d31 Compare December 2, 2024 11:45
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.

@mihaly-dombi small addition so I can trigger tests and validate your change.

@@ -13,7 +13,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: c6b0a29e-1da9-4512-9002-7bfd0cba2246
dockerImageTag: 6.2.1
dockerImageTag: 6.3.0
Copy link
Member

Choose a reason for hiding this comment

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

You need also to add an entry in the docs changelog.

@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Dec 17, 2024
@marcosmarxm
Copy link
Member

marcosmarxm commented Dec 17, 2024

/format-fix

Format-fix job started... Check job output.

🟦 Job completed successfully (no changes).

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/amazon-ads
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants