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

fix(in-app-purchase2): Registering a product with an alias is now optional #2946

Merged
merged 1 commit into from
Apr 7, 2019

Conversation

tgirgin23
Copy link
Contributor

According to the documentation, alias is optional
https://github.com/j3k0/cordova-plugin-purchase/blob/master/doc/api.md#storeregisterproduct

Short Description of the issue:
It is currently not possible to register a product without an alias using the ionic-native npm package for cordova-plugin-purchase.

Steps to reproduce:
Try registering a product without an alias throws the following error:
error TS2345: Argument of type '{ id: any; type: string; }' is not assignable to parameter of type 'IAPProductOptions'. [ng] Property 'alias' is missing in type '{ id: any; type: string; }'.

@tgirgin23 tgirgin23 changed the title Making the alias of IAPProductOptions optional [in-app-purchase2] Registering a product with an alias should be optional Mar 4, 2019
@tgirgin23 tgirgin23 changed the title [in-app-purchase2] Registering a product with an alias should be optional fix(in-app-purchase2): Registering a product with an alias is now optional Mar 29, 2019
@danielsogl danielsogl added the target: patch This PR is targeted for the next patch release label Apr 7, 2019
@danielsogl danielsogl merged commit 4e60c63 into danielsogl:master Apr 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants