Bug fixes for various examples. (googleads#358)

BenRKarl · web-flow · commit fe6b85290c9c · 2020-12-14T16:30:38.000-05:00
diff --git a/examples/hotel_ads/add_hotel_ad.py b/examples/hotel_ads/add_hotel_ad.py
@@ -28,6 +28,9 @@
 import google.ads.google_ads.client
 
 
+_DEFAULT_CPC_BID_CEILING_MICRO_AMOUNT = 20000000
+
+
 def main(
     client, customer_id, hotel_center_account_id, bid_ceiling_micro_amount
 ):
@@ -267,8 +270,12 @@ def add_hotel_campaign(
         "-b",
         "--bid_ceiling_micro_amount",
         type=int,
-        required=True,
-        help=("The bid ceiling micro amount for " "the hotel campaign."),
+        required=False,
+        default=_DEFAULT_CPC_BID_CEILING_MICRO_AMOUNT,
+        help=(
+            "The bid ceiling micro amount for the hotel campaign. "
+            f"The default value is {_DEFAULT_CPC_BID_CEILING_MICRO_AMOUNT}"
+        ),
     )
     parser.add_argument(
         "-a",
diff --git a/examples/hotel_ads/add_hotel_ad_group_bid_modifiers.py b/examples/hotel_ads/add_hotel_ad_group_bid_modifiers.py
@@ -61,7 +61,7 @@ def main(client, customer_id, ad_group_id):
 
     # Add the bid modifiers
     try:
-        ag_bm_response = ag_bm_service.mutate_ad_groups(
+        ag_bm_response = ag_bm_service.mutate_ad_group_bid_modifiers(
             customer_id, [check_in_ag_bm_operation, los_ag_bm_operation]
         )
     except google.ads.google_ads.errors.GoogleAdsException as ex:
diff --git a/examples/hotel_ads/add_hotel_listing_group_tree.py b/examples/hotel_ads/add_hotel_listing_group_tree.py
@@ -486,8 +486,8 @@ def _create_ad_group_criterion(
         required=True,
         help="Specify the CPC bid micro amount to be set on a created ad group "
         "criterion. For simplicity, each ad group criterion will use the below "
-        "amount equally. In practice, you probably want to use different values "
-        "for each ad group criterion.",
+        "amount equally. In practice, you probably want to use different "
+        "values for each ad group criterion.",
     )
     args = parser.parse_args()
 
diff --git a/examples/shopping_ads/add_listing_scope.py b/examples/shopping_ads/add_listing_scope.py
@@ -58,10 +58,12 @@ def main(client, customer_id, campaign_id):
     product_brand_dimension.product_brand.value = "google"
 
     product_custom_attribute_dimension = dimensions.add()
-    product_custom_attribute_dimension.index = client.get_type(
-        "ProductCustomAttributeIndexEnum", version="v6"
-    ).INDEX0
-    product_custom_attribute_dimension.value = "top_selling_products"
+    product_custom_attribute_dimension.product_custom_attribute.index = (
+        client.get_type("ProductCustomAttributeIndexEnum", version="v6").INDEX0
+    )
+    product_custom_attribute_dimension.product_custom_attribute.value = (
+        "top_selling_products"
+    )
 
     product_type_level_enum = client.get_type(
         "ProductTypeLevelEnum", version="v6"

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ def main(client, customer_id, ad_group_id):`
`61`	`61`
`62`	`62`	`# Add the bid modifiers`
`63`	`63`	`try:`
`64`		`- ag_bm_response = ag_bm_service.mutate_ad_groups(`
	`64`	`+ ag_bm_response = ag_bm_service.mutate_ad_group_bid_modifiers(`
`65`	`65`	`customer_id, [check_in_ag_bm_operation, los_ag_bm_operation]`
`66`	`66`	`)`
`67`	`67`	`except google.ads.google_ads.errors.GoogleAdsException as ex:`