Skip to content

Fix one more dynamic import needed for vendored-in google ads#30564

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-one-more-dynamic-google-ads-import
Apr 10, 2023
Merged

Fix one more dynamic import needed for vendored-in google ads#30564
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-one-more-dynamic-google-ads-import

Conversation

@potiuk

@potiuk potiuk commented Apr 10, 2023

Copy link
Copy Markdown
Member

Continuation of #30544


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@potiuk potiuk force-pushed the fix-one-more-dynamic-google-ads-import branch from dcedb1a to a33310e Compare April 10, 2023 12:20
@potiuk

potiuk commented Apr 10, 2023

Copy link
Copy Markdown
Member Author

cc: @lwyszomi @cgadam @vanducng @eladkal #30553 - we will need RC2 for Google as well. This is one more change that @cgadam found when testing RC2 (This is to handle a case when there is an error returned by google ads and it returns additional metadata).

@potiuk potiuk merged commit 18ec7f2 into apache:main Apr 10, 2023
@potiuk potiuk deleted the fix-one-more-dynamic-google-ads-import branch April 10, 2023 15:49
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.

2 participants