Skip to content

Add new Line Item fields for MAP objectives#264

Closed
osowskit wants to merge 4 commits intoxdevplatform:masterfrom
osowskit:master
Closed

Add new Line Item fields for MAP objectives#264
osowskit wants to merge 4 commits intoxdevplatform:masterfrom
osowskit:master

Conversation

@osowskit
Copy link
Contributor

Adding new iOS and Android app store identifiers on the line items endpoint. Announcement. This is part of the line_item_apps deprecation and that endpoint will be removed mid-Sept.

Note existing values set on the line_item_apps will automatically be used by the line_items endpoint.

closes #263

@osowskit
Copy link
Contributor Author

I still need to bump the version

@osowskit
Copy link
Contributor Author

@tushdante could you help get this merged and release a new SDK?

@osowskit osowskit requested a review from tushdante September 21, 2020 16:04
Copy link
Contributor

@tushdante tushdante left a comment

Choose a reason for hiding this comment

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

Might wanna check our tests to verify if things need to be updated there as well, but otherwise LGTM

@osowskit osowskit changed the base branch from master to tosowski/line_item_apps September 24, 2020 21:54
@osowskit osowskit changed the base branch from tosowski/line_item_apps to master September 24, 2020 21:55
@osowskit
Copy link
Contributor Author

Cross fork CI builds won't work. Closing this and following up in #266

@osowskit osowskit closed this Sep 24, 2020
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.

New MAP LineItem fields

2 participants