-
Notifications
You must be signed in to change notification settings - Fork 441
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
Upgrade AppAuth-iOS to 1.0.0 #338
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks for taking care if it works with autolinking :)
README.md
Outdated
|
||
Then run `pod install`. Note that version 0.94 is the first of the library to support iOS 11. | ||
Then run `pod install`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, all instructions about linking on both paltforms are obsolete in RN 0.60 if the library works with autolinking. All you need to document now is:
- running
pod install
after adding the dep - how to use with Carthage or anything that is not CocoaPods or manual link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, I'll update the instructions. Thanks for all your help on this!
1c32f12
to
ad05491
Compare
Note - next major release will have RN0.60 as a peer dependency, so the instructions for <RN0.60 will be removed before then. I'm leaving both instructions in (for now), for users who are installing the library between now and the next major release, and are using the instructions from the master branch on GitHub. |
Released in v5.0.0 🎉 |
Fixes #337
Fixes #339
Description
Upgrades the native AppAuth-iOS dependency.
Note: this will have to be a major release, due to a typo fix in a method name