Skip to content

Update Libraries #81

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

Merged
merged 2 commits into from
Oct 20, 2015
Merged

Update Libraries #81

merged 2 commits into from
Oct 20, 2015

Conversation

wangmengyan95
Copy link
Contributor

  1. Update FB Utils to open source 1.10.3
  2. Update Twitter Utils to open source 1.10.3
  3. Update Parse to 1.10.3
  4. Update FB to 4.6.0
  5. Update Android Support Library to 23.0.1
  6. Delete FB Utils and Twitter Utils jar
  7. Update README.MD

@wangmengyan95 wangmengyan95 force-pushed the wangmengyan.update_libraries branch from a88850b to c843bcb Compare October 8, 2015 22:01
@wangmengyan95 wangmengyan95 force-pushed the wangmengyan.update_libraries branch from c843bcb to 32b725d Compare October 8, 2015 22:03
@@ -18,13 +18,13 @@ ext {
compileSdkVersion = 21
buildToolsVersion = "21.1.1"

minSdkVersion = 9
minSdkVersion = 15
Copy link
Contributor

Choose a reason for hiding this comment

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

what's the reason for this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

FB SDK requires this.
Actually it is a requirement for 4.6.0, if we use 4.4.0 then it is fine to keep it 9.
What do you think?

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this variable only used for the samples? It looks like ParseUI-Login and ParseUI-Widget still have minSdkVersion = 9. If this is the case, I think it's fine.

@grantland
Copy link
Contributor

LGTM, we'll be targeting FB SDK 4.4.0 for minSdkVersion 9 compatibility, but 4.6.0 for our samples.

wangmengyan95 added a commit that referenced this pull request Oct 20, 2015
@wangmengyan95 wangmengyan95 merged commit bd5e917 into master Oct 20, 2015
@wangmengyan95 wangmengyan95 deleted the wangmengyan.update_libraries branch October 20, 2015 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants