Skip to content
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

Timob 9999: Clean up Android platform code #2876

Merged
merged 9 commits into from Sep 11, 2012
Merged

Conversation

hieupham007
Copy link
Contributor

This PR removes dead code and deprecated code prior to 1.8.0.
Testing steps in JIRA: https://jira.appcelerator.org/browse/TIMOB-9999

@ayeung
Copy link
Contributor

ayeung commented Sep 7, 2012

Will CR + FR

@@ -207,17 +206,6 @@ protected void doSetBackgroundImage(Object image)
}
}

@Kroll.setProperty(runOnUiThread=true) @Kroll.method(runOnUiThread=true)
public void setOrientation(int tiOrientationMode)
Copy link
Contributor

Choose a reason for hiding this comment

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

If we remove this property, the docs need to be updated accordingly as well. The state in the docs should be changed from 'deprecated' to 'removed'

@ayeung
Copy link
Contributor

ayeung commented Sep 7, 2012

Code reviewed. Before continuing with this PR, please test this against modules i.e. create a new module with this and run older modules with this change. This should be added as a part of the test instructions as well. I also mentioned that the docs need to be updated to the 'removed' state for APIs that transition from 'deprecated' to 'removed'.

@hieupham007
Copy link
Contributor Author

Code updated. Also ran ti.cloud + create new module and run it against PR. Ready for re-review.

@ayeung
Copy link
Contributor

ayeung commented Sep 11, 2012

Code reviewed and functionally tested. Ran anvil, built the sdk, and tested modules. Request Accepted.

ayeung pushed a commit that referenced this pull request Sep 11, 2012
Timob 9999: Clean up Android platform code
@ayeung ayeung merged commit a2e07ec into tidev:master Sep 11, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants