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

Conversation

@YukiOya
Copy link
Contributor

@YukiOya YukiOya commented Jul 9, 2019

Description

I added a converting process because UserInfo of NSError contains a value of a type not supported by flutter like NSError and NSURL.

https://flutter.dev/docs/development/platform-integration/platform-channels#platform-channel-data-types-support-and-codecs

Related Issues

flutter/flutter#35554

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
  • All existing and new tests are passing.
  • I updated/added relevant documentation (doc comments with ///).
  • The analyzer (flutter analyze) does not report any problems on my PR.
  • I read and followed the Flutter Style Guide.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
  • I updated CHANGELOG.md to add a description of the change.
  • I signed the CLA.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
  • No, this is not a breaking change.

Copy link
Contributor

@cyanglaz cyanglaz left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution! This is great!

@@ -1,3 +1,7 @@
## 0.2.0+4

* Fix a UserInfo types on NSError.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nits:

Suggested change
* Fix a UserInfo types on NSError.
* iOS: Support unsupported UserInfo value types on NSError.

Copy link
Contributor Author

@YukiOya YukiOya Jul 10, 2019

Choose a reason for hiding this comment

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

@cyanglaz
Hi. Thank you for your quick review.
I rebase master and fixed it ab59ee9 .

@cyanglaz cyanglaz self-assigned this Jul 9, 2019
@cyanglaz cyanglaz added the submit queue The Flutter team is in the process of landing this PR. label Jul 9, 2019
@cyanglaz
Copy link
Contributor

@YukiOya There seems to be some conflicts. Could you please rebase?

@YukiOya YukiOya changed the title [in_app_purchase] [iOS] Fix UserInfo types on NSError [in_app_purchase] [iOS] Support unsupported UserInfo value types on NSError. Jul 10, 2019
@YukiOya YukiOya changed the title [in_app_purchase] [iOS] Support unsupported UserInfo value types on NSError. [in_app_purchase] iOS: Support unsupported UserInfo value types on NSError. Jul 10, 2019
@doctorlh
Copy link

great

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes submit queue The Flutter team is in the process of landing this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants