Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Conversation

@cyanglaz
Copy link
Contributor

@cyanglaz cyanglaz commented Feb 5, 2019

To make to code simple, NSLocale has not been fully exposed.
Expose the NSLocale inside the priceLocale layer to match the SKProduct composition; this makes future changes regarding priceLocale to be non-breaking changes.

@cyanglaz cyanglaz requested a review from mklim February 5, 2019 23:04

- (nullable NSDictionary *)toMap {
NSMutableDictionary *map = [[NSMutableDictionary alloc] init];
if (@available(iOS 10.0, *)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

So do we not have access to this information on ios < 10?

Copy link
Contributor

@mklim mklim left a comment

Choose a reason for hiding this comment

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

LGTM

@cyanglaz cyanglaz merged commit a100fbf into flutter:master Feb 6, 2019
andreidiaconu pushed a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
andreidiaconu added a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
karantanwar pushed a commit to karantanwar/plugins that referenced this pull request Feb 19, 2019
karantanwar pushed a commit to karantanwar/plugins that referenced this pull request Feb 19, 2019
@cyanglaz cyanglaz changed the title Expose nslocale and expose currencySymbol instead of currencyCode to match android [In_app_purchase] Expose nslocale and expose currencySymbol instead of currencyCode to match android Feb 22, 2019
@cyanglaz cyanglaz deleted the iap_expose_nslocale branch March 4, 2019 21:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants