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

Source Amazon Ads: added adId to product report stream #11660

Merged
merged 8 commits into from
Apr 26, 2022
Merged

Source Amazon Ads: added adId to product report stream #11660

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
01c3c88
added adId to product report stream
agroh1 Apr 1, 2022
376c56f
Merge branch 'airbytehq:master' into issue-11659
agroh1 Apr 4, 2022
00ccf78
simplified removal of adId
agroh1 Apr 8, 2022
44ae708
removed duplicate comment
agroh1 Apr 8, 2022
b9c7dc1
Bumped dockerfile version
agroh1 Apr 8, 2022
46ddcf7
Merge branch 'issue-11659' of github.com:agroh1/airbyte into marcos/t…
marcosmarxm Apr 20, 2022
5f87976
bump connector version
marcosmarxm Apr 20, 2022
0dfa74a
Merge branch 'master' into marcos/test-pr-11659
marcosmarxm Apr 21, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion airbyte-config/init/src/main/resources/seed/source_definitions.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
- name: Amazon Ads
sourceDefinitionId: c6b0a29e-1da9-4512-9002-7bfd0cba2246
dockerRepository: airbyte/source-amazon-ads
dockerImageTag: 0.1.5
dockerImageTag: 0.1.6
documentationUrl: https://docs.airbyte.io/integrations/sources/amazon-ads
icon: amazonads.svg
sourceType: api
Expand Down
2 changes: 1 addition & 1 deletion airbyte-config/init/src/main/resources/seed/source_specs.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@
supportsNormalization: false
supportsDBT: false
supported_destination_sync_modes: []
- dockerImage: "airbyte/source-amazon-ads:0.1.5"
- dockerImage: "airbyte/source-amazon-ads:0.1.6"
spec:
documentationUrl: "https://docs.airbyte.io/integrations/sources/amazon-ads"
connectionSpecification:
Expand Down
2 changes: 1 addition & 1 deletion airbyte-integrations/connectors/source-amazon-ads/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,5 @@ RUN pip install .
ENV AIRBYTE_ENTRYPOINT "python /airbyte/integration_code/main.py"
ENTRYPOINT ["python", "/airbyte/integration_code/main.py"]

LABEL io.airbyte.version=0.1.5
LABEL io.airbyte.version=0.1.6
LABEL io.airbyte.name=airbyte/source-amazon-ads
8 changes: 8 additions & 0 deletions .../connectors/source-amazon-ads/source_amazon_ads/streams/report_streams/products_report.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,7 @@
"campaignId",
"adGroupName",
"adGroupId",
"adId",
"impressions",
"clicks",
"cost",
Expand Down Expand Up @@ -156,6 +157,7 @@
"adGroupId",
"keywordId",
"keywordText",
"adId",
"asin",
"otherAsin",
"sku",
Expand All @@ -179,6 +181,7 @@
"campaignId",
"adGroupName",
"adGroupId",
"adId",
"asin",
"otherAsin",
"sku",
Expand Down Expand Up @@ -262,4 +265,9 @@ def _get_init_report_body(self, report_date: str, record_type: str, profile):
if profile.accountInfo.type == "vendor":
metrics_list = copy(metrics_list)
metrics_list.remove("sku")

# adId is automatically added to the report by amazon and requesting adId causes an amazon error
if "adId" in metrics_list:
metrics_list.remove("adId")

return {**body, "metrics": ",".join(metrics_list)}
1 change: 1 addition & 0 deletions docs/integrations/sources/amazon-ads.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ Start date used for generating reports starting from the specified start date. S

| Version | Date | Pull Request | Subject |
| :--- | :--- | :--- | :--- |
| `0.1.6` | 2022-04-20 | [\#11659](https://github.com/airbytehq/airbyte/pull/11659) | Add adId to products report |
| `0.1.5` | 2022-04-08 | [\#11430](https://github.com/airbytehq/airbyte/pull/11430) | `Added support OAuth2.0` |
| `0.1.4` | 2022-02-21 | [\#10513](https://github.com/airbytehq/airbyte/pull/10513) | `Increasing REPORT_WAIT_TIMEOUT for supporting report generation which takes longer time ` |
| `0.1.3` | 2021-12-28 | [\#8388](https://github.com/airbytehq/airbyte/pull/8388) | `Add retry if recoverable error occured for reporting stream processing` |
Expand Down