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

Fixed consumable products reading on Android #439

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

33-Elephants
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jul 4, 2023

Codecov Report

Merging #439 (5a92914) into main (980cbc8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #439   +/-   ##
=======================================
  Coverage   47.69%   47.69%           
=======================================
  Files           4        4           
  Lines         478      478           
=======================================
  Hits          228      228           
  Misses        250      250           

Copy link
Owner

@hyochan hyochan left a comment

Choose a reason for hiding this comment

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

@33-Elephants Thank you for your contribution! Could you kindly explain how does it fixes problem?

@33-Elephants
Copy link
Contributor Author

As far as I remember there was a problem with reading "price", "currency" and "localizedPrice" fields for one-time charge items. Reading product info from subscriptionOfferDetails field works for subscriptions only. For one-time charge products oneTimePurchaseOfferDetails must be used instead.

@hyochan hyochan force-pushed the main branch 2 times, most recently from 45035a3 to 475e88d Compare October 31, 2023 17:32
Copy link
Owner

@hyochan hyochan left a comment

Choose a reason for hiding this comment

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

This seems valid and focuses on a key aspect of handling pricing and product details for different types of in-app purchases in Android, particularly distinguishing between one-time purchases and subscriptions 🚀

@hyochan hyochan merged commit 7518661 into hyochan:main Oct 31, 2023
@hyochan hyochan added enhancement New feature or request and removed waiting for customer response labels Oct 31, 2023
JoyHong pushed a commit to JoyHong/flutter_inapp_purchase that referenced this pull request Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android android related issue enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants