Skip to content

Commit

Permalink
🐛 Source Google Ads: Disable raising error for not enabled accounts (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
tolik0 committed Jan 12, 2024
1 parent 94dc686 commit c967f14
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: 253487c0-2246-43ba-a21f-5116b20a2c50
dockerImageTag: 3.2.0
dockerImageTag: 3.2.1
dockerRepository: airbyte/source-google-ads
documentationUrl: https://docs.airbyte.com/integrations/sources/google-ads
githubIssueLabel: source-google-ads
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,6 @@ class CustomerClient(GoogleAdsStream):
Customer Client stream: https://developers.google.com/google-ads/api/fields/v15/customer_client
"""

CATCH_CUSTOMER_NOT_ENABLED_ERROR = False
primary_key = ["customer_client.id"]

def __init__(self, customer_status_filter: List[str], **kwargs):
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,15 +34,6 @@ def mock_get_customers(mocker):
["CUSTOMER_NOT_FOUND"],
"Failed to access the customer '123'. Ensure the customer is linked to your manager account or check your permissions to access this customer account.",
),
(
["CUSTOMER_NOT_ENABLED"],
(
"The customer account '123' hasn't finished signup or has been deactivated. "
"Sign in to the Google Ads UI to verify its status. "
"For reactivating deactivated accounts, refer to: "
"https://support.google.com/google-ads/answer/2375392."
),
),
(["QUERY_ERROR"], "Incorrect custom query. Error in query: unexpected end of query."),
(
["RESOURCE_EXHAUSTED"],
Expand Down
1 change: 1 addition & 0 deletions docs/integrations/sources/google-ads.md
Original file line number Diff line number Diff line change
Expand Up @@ -280,6 +280,7 @@ Due to a limitation in the Google Ads API which does not allow getting performan

| Version | Date | Pull Request | Subject |
|:---------|:-----------|:---------------------------------------------------------|:-------------------------------------------------------------------------------------------------------------------------------------------------------|
| `3.2.1` | 2024-01-12 | [34200](https://github.com/airbytehq/airbyte/pull/34200) | Disable raising error for not enabled accounts |
| `3.2.0` | 2024-01-09 | [33707](https://github.com/airbytehq/airbyte/pull/33707) | Add possibility to sync all connected accounts |
| `3.1.0` | 2024-01-09 | [33603](https://github.com/airbytehq/airbyte/pull/33603) | Fix two issues in the custom queries: automatic addition of `segments.date` in the query; incorrect field type for `DATE` fields. |
| `3.0.2` | 2024-01-08 | [33494](https://github.com/airbytehq/airbyte/pull/33494) | Add handling for 401 error while parsing response. Add `metrics.cost_micros` field to Ad Group stream. |
Expand Down

0 comments on commit c967f14

Please sign in to comment.