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

Updates to switch to ios for Cards #252

Merged
merged 4 commits into from
Nov 5, 2021
Merged

Conversation

osowskit
Copy link
Contributor

We recently introduced a validation rule that requires a match between the ids set on the creative level and those on the line_item level.

With that, we have made a change to help avoid any issues with the validation rule by reducing the parameters available. Specifically:

  • For App Cards:
    • Deprecated ipad_app_id parameter
    • Renamed iphone_app_id to ios_app_store_identifier parameters
  • Renamed iphone_deep_link to ios_deep_link
  • Moving forward, switch to ios_app_store_identifier to align with the line_item field naming.

This update was related to the Line Item/Ad Group Mobile App Promotion changes introduced in v9.

@osowskit osowskit requested a review from tushdante September 10, 2021 16:19
@osowskit osowskit merged commit 5a1dba7 into ads-api-v10 Nov 5, 2021
@osowskit osowskit deleted the osowskit/aps-api-v10 branch November 5, 2021 18:24
tushdante added a commit that referenced this pull request Nov 18, 2021
* Updates to switch to ios for Cards (#252)

* Removing deprecated field salt

* Removing deprecated field salt in tests

* Add enum for Pay By Impression

* renamed/removed line item props

* moved props out of beta

* updated enums

* added support for cards endpoint

* removed old tailored audiences endpoint

* added new cards endpoint with examples

* updated tailored audiences to custom audiences

* update audience_summary to audience_estimate

* removed automatically_select_bid from line items

* Remove deprecated tracking_tags field from line items

* Add Tracking Tags support

Co-authored-by: Tushar Bhushan <[email protected]>
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.

1 participant