Skip to content

V4 API updates: added some more changes#1010

Merged
NachoSoto merged 3 commits into
mainfrom
migration-fixes
Dec 1, 2021
Merged

V4 API updates: added some more changes#1010
NachoSoto merged 3 commits into
mainfrom
migration-fixes

Conversation

@NachoSoto

@NachoSoto NachoSoto commented Nov 30, 2021

Copy link
Copy Markdown
Contributor

Changes:

  • Added note about Carthage integration
  • Added missing RevenueCat.ErrorCode.operationAlreadyInProgressForProductError to Swift section
  • Added Package.product
  • Fixed : in parameter lists
  • Fixed PurchasesDelegate method name
  • Added error handling section
  • Added createAlias/identify/reset deprecations ([sc-11512])

TODO:

Package.product -> Package.productDetails is now documented here, but this opens another can of worms because of the introduction of ProductDetails:

- Added note about `Carthage` integration
- Added missing `RevenueCat.ErrorCode.operationAlreadyInProgressForProductError` to Swift section
- Added `Package.product`
- Fixed `:` in parameter lists
- Fixed `PurchasesDelegate` method name
- Added error handling section
Comment thread docs/V4_API_Updates.md
</tbody>
</table>

### Swift Error handling

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧡

@shortcut-integration

Copy link
Copy Markdown

This pull request has been linked to Shortcut Story #11512: update migration guide to include removal of identity v2.

@NachoSoto NachoSoto marked this pull request as ready for review December 1, 2021 17:15
@NachoSoto

Copy link
Copy Markdown
Contributor Author

I created one issue for the only missing thing in the TODO list, so I marked this ready for review for now.

@taquitos taquitos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

📜 ❤️ 🐐

@NachoSoto NachoSoto merged commit 29c2070 into main Dec 1, 2021
@NachoSoto NachoSoto deleted the migration-fixes branch December 1, 2021 19:38
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