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

Rename get product category constants #474

Closed
wants to merge 3 commits into from

Conversation

danielfrg
Copy link
Contributor

No description provided.

Change-Id: Ibe38e3a124318893ca4917e388c0ed9cf73a4f06
* 'main' of github.com:googleads/google-ads-ruby:
  Demonstrate passing consent in offline conversion upload (#473)
  Update README.md (#472)

Change-Id: I98c5325a7772bf0a49d166d8152849fd3ccf4041
Change-Id: I17867fc4eae63c6e7b36f4f2720b322df65583da
@danielfrg danielfrg requested a review from a team as a code owner January 12, 2024 16:02
@danielfrg danielfrg closed this Jan 12, 2024
# values. This is an example to override a recommended ad when a
# TextAdRecommendation is applied.
# For details, please read
# https://developers.google.com/google-ads/api/reference/rpc/google.ads.google_ads.v1.services#google.ads.google_ads.v1.services.ApplyRecommendationOperation
Copy link
Member

Choose a reason for hiding this comment

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

This link is a 404

@@ -70,10 +70,46 @@ def detect_and_apply_recommendations(customer_id)
end
end

# [START apply_recommendation]
def apply_recommendation(client, customer_id, recommendation_resource)
# If you have a recommendation_id instead of the resournce_name
Copy link
Member

Choose a reason for hiding this comment

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

typo in resource_name

@@ -33,34 +33,34 @@ def detect_and_apply_recommendations(customer_id)
client = Google::Ads::GoogleAds::GoogleAdsClient.new

query = <<~QUERY
SELECT recommendation.resource_name
SELECT recommendation.resource_name, recommendation.campaign,
recommendation.keyword_recommendation
Copy link
Member

Choose a reason for hiding this comment

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

Can we put one field per line?

end

apply_recommendation(client, customer_id, recommendation.resource_name)
end

if i < NUMBER_OF_RUNS - 1
Copy link
Member

Choose a reason for hiding this comment

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

Why does this still exist if we're reducing the number of runs to 1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, ignore this changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants