liveservices / LiveSDK-for-iOS

LiveSDK library for integrating with Live Connect
MIT License
138 stars 84 forks source link

Allow the user to avoid the local plist storage for refreshToken + correct download approach #44

Closed danielgindi closed 9 years ago

danielgindi commented 9 years ago

I do not know how to build this framework, there's no build script. So someone else will have to do that.

danielgindi commented 9 years ago

These are some important improvements... Are you planning on pulling the request?

rgregg commented 9 years ago

Thanks for the pull request @danielgindi! We're in the middle of some refactoring of the LiveSDK for iOS to support cocoapods and clean up what's in the repo. I noticed you have several changes integrated into a single pull request, including: a) download to disk instead of RAM, b) enable setting/restoring sessions, c) delegate-based refresh token storage. It would be helpful for us if you can split these into individual pull requests so we can evaluate each change individually.

danielgindi commented 9 years ago

Well I actually have no idea how to separate pull requests on GitHub... they are already on different commits

danielgindi commented 9 years ago

The longer you wait before pulling the request - the more commits are added ;-) I think GitHub can't create separate pull requests

rgregg commented 9 years ago

You can only have one pull request per branch. You need to separate the changes into different feature branches and submit a pull request from each branch.

danielgindi commented 9 years ago

I've separated them - thanks!

danielgindi commented 9 years ago

Anyone alive here? :-)

rgregg commented 9 years ago

There aren't a lot of engineers around right now due to the holidays. We'll get to your pull requests as soon as we can.