Skip to content
This repository has been archived by the owner on May 1, 2020. It is now read-only.

Downgrade API 14 #61

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

ppamorim
Copy link
Contributor

I created this new branch only to downgrade de API 14. Waiting the verify and merge of #58

A better code structure and API downgraded to 14
I have changed a lot of things there, the most one is about
compatibility with API 14
Changes on Example structure and some corrections at PlaygroundActivity
@@ -2,6 +2,8 @@ apply plugin: 'com.android.application'

dependencies {
compile project(':rebound-android')
compile 'com.android.support:appcompat-v7:22.2.0'
compile 'com.jakewharton:butterknife:7.0.0'
Copy link
Contributor

Choose a reason for hiding this comment

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

remove

@willbailey
Copy link
Contributor

Still need to remove the butterknife dependency if you want to land this

@ppamorim
Copy link
Contributor Author

@willbailey This will be automatically removed when you merge #58? If not, sorry for this issues...

@ppamorim
Copy link
Contributor Author

@willbailey Well, I'll do that change, but isn't better to wait to merge #58 firstly?

@willbailey
Copy link
Contributor

I think landing the API downgrade first is the way to go. I'm not sure I understand the goal of #58, but lowering the API bar does seem like a nice improvement.

@ghost ghost added the CLA Signed label Jul 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants