[in_app_purchase] Update minimum Flutter version to 3.3 and iOS 11#6873
[in_app_purchase] Update minimum Flutter version to 3.3 and iOS 11#6873auto-submit[bot] merged 4 commits intoflutter:mainfrom
Conversation
jmagman
force-pushed
the
iap-bump-version-33
branch
from
07cffb1 to
4f941a4
Compare
| <string>1.0</string> | ||
| <key>MinimumOSVersion</key> | ||
| <string>8.0</string> | ||
| <string>11.0</string> |
There was a problem hiding this comment.
The changes in packages/in_app_purchase/in_app_purchase/example/ and the Xcode project files/Podfile were automatically applied by the Flutter tool.
| NSLocale *system = [[NSLocale alloc] initWithLocaleIdentifier:@"en_US"]; | ||
| NSDictionary *map = [FIAObjectTranslator getMapFromNSLocale:system]; | ||
| XCTAssertEqualObjects(map[@"currencySymbol"], system.currencySymbol); | ||
| XCTAssertEqualObjects(map[@"countryCode"], system.countryCode); |
There was a problem hiding this comment.
I edited this manually, removing the @available check.
| s.ios.dependency 'Flutter' | ||
| s.osx.dependency 'FlutterMacOS' | ||
| s.ios.deployment_target = '9.0' | ||
| s.ios.deployment_target = '11.0' |
There was a problem hiding this comment.
I also edited this manually.
jmagman
force-pushed
the
iap-bump-version-33
branch
from
4f941a4 to
0d70fcd
Compare
| @@ -1,3 +1,7 @@ | |||
| ## 0.3.5 | |||
There was a problem hiding this comment.
@stuartmorgan I bumped the iOS federated version by a minor version for the iOS minimum change, wdyt?
There was a problem hiding this comment.
Seems reasonable. There's a lot of grey area in minor vs bugfix.
jmagman
changed the title
| @@ -1,3 +1,7 @@ | |||
| ## NEXT | |||
There was a problem hiding this comment.
We should do a version bump and README update here, as with ios_platform_images.
(It's arguable in the federated world what "supported version" actually means, since technically you could use 3.1.1 or whatever with iOS 9 and an older version of Flutter and the resolver would just give you an older version of the implementation package, but I think having the current implementation's version in the main package is the least confusing.)
There was a problem hiding this comment.
You're right, not sure how I missed this chart in the README:
Will add a patch bump.
| version: 0.3.5 | ||
|
|
||
| environment: | ||
| sdk: ">=2.14.0 <3.0.0" |
There was a problem hiding this comment.
The next line (I can't comment there since it's outside the context) is missing the flutter minimum update to 3.3.
There was a problem hiding this comment.
🤦♀️ this was hard to keep all these package changes organized, thanks for catching. Glad I split it into reviewable-sized changes.
jmagman
force-pushed
the
iap-bump-version-33
branch
from
0d70fcd to
4192d3c
Compare
jmagman
added
the
autosubmit
* cd09d9d31 [ci] Update iOS simulator (flutter/plugins#7131) * 016c3b7f1 Roll Flutter from df41e58 to 22e17bb (28 revisions) (flutter/plugins#7186) * 7160f55e8 [ios_platform_images] Update minimum version to iOS 11 (flutter/plugins#6874) * ea048a249 [in_app_purchase] Update minimum Flutter version to 3.3 and iOS 11 (flutter/plugins#6873) * 530442456 [google_sign_in_web] Migrate to the GIS SDK. (flutter/plugins#6921) * 9a3a77e6c [image_picker] Fix images changing to incorrect orientation (flutter/plugins#7187) * 8f3419be5 Roll Flutter from 22e17bb to 298d8c7 (20 revisions) (flutter/plugins#7190)
3 participants
in_app_purchasepart of flutter/flutter#101965.Built the
in_app_purchaseandin_app_purchase_iosexample projects on Flutter 3.3. and let the tool auto-migrate various Xcode project/Podfile files.Pre-launch Checklist
dart format.)[shared_preferences]pubspec.yamlwith an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.mdto add a description of the change, following repository CHANGELOG style.///).If you need help, consider asking for advice on the #hackers-new channel on Discord.