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

Feature android lib #1030

Merged
merged 19 commits into from
Oct 14, 2014
Merged

Conversation

m4gr3d
Copy link
Member

@m4gr3d m4gr3d commented Sep 1, 2014

Extract the common android resources into an android library project.

This allows to avoid logic and resources reimplementation for android related projects (i.e: main app, wearables app).

Also serves as a base for decoupling of the android front end from the back end.

Setup, and tested in Android Studio, and Eclipse.

@m4gr3d m4gr3d added the refactor label Sep 1, 2014
@m4gr3d
Copy link
Member Author

m4gr3d commented Sep 1, 2014

@arthurbenemann I tested this, and it works great. Just awaiting your code review.

@arthurbenemann
Copy link
Member

@ne0fhyk I'm not sure what you where trying to accomplish here, can you give more details? If we can avoid creating another library that would be good.

@m4gr3d
Copy link
Member Author

m4gr3d commented Sep 15, 2014

@arthurbenemann the library project stores all the common resources used by the different android projects (wear, app): http://stackoverflow.com/a/24592107

The alternative would be to duplicate resources, and code for each project.

…into feature_android_lib

Conflicts:
	Android/project.properties
	Android/res/drawable-hdpi/ic_loiter.png
	Android/res/drawable-hdpi/ic_pause.png
	Android/res/drawable-mdpi/ic_loiter.png
	Android/res/drawable-mdpi/ic_pause.png
	Android/res/drawable-xhdpi/.DS_Store
	Android/res/drawable-xhdpi/ic_loiter.png
	Android/res/drawable-xhdpi/ic_pause.png
	Android/res/drawable/ic_action_quad.xml
	Android/res/drawable/ic_navigate.xml
	Android/src/org/droidplanner/android/maps/providers/google_map/GoogleMapFragment.java
	Android/src/org/droidplanner/android/notifications/EmergencyBeepNotificationProvider.java
	AndroidLib/res/drawable-hdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-hdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable-hdpi/ic_loiter.png
	AndroidLib/res/drawable-land/mission_control_bg.xml
	AndroidLib/res/drawable-mdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-mdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable-mdpi/ic_loiter.png
	AndroidLib/res/drawable-port/mission_control_bg.xml
	AndroidLib/res/drawable-sw540dp/mission_control_bg.xml
	AndroidLib/res/drawable-xhdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-xhdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable-xhdpi/ic_loiter.png
	AndroidLib/res/drawable-xxhdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-xxhdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable/flight_control_button_selector.xml
	AndroidLib/res/drawable/ic_action_quad.xml
…into feature_android_lib

Conflicts:
	Android/project.properties
	Android/res/drawable-hdpi/ic_loiter.png
	Android/res/drawable-hdpi/ic_pause.png
	Android/res/drawable-mdpi/ic_loiter.png
	Android/res/drawable-mdpi/ic_pause.png
	Android/res/drawable-xhdpi/.DS_Store
	Android/res/drawable-xhdpi/ic_loiter.png
	Android/res/drawable-xhdpi/ic_pause.png
	Android/res/drawable/ic_action_quad.xml
	Android/res/drawable/ic_navigate.xml
	Android/src/org/droidplanner/android/maps/providers/google_map/GoogleMapFragment.java
	Android/src/org/droidplanner/android/notifications/EmergencyBeepNotificationProvider.java
	AndroidLib/res/drawable-hdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-hdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable-hdpi/ic_loiter.png
	AndroidLib/res/drawable-land/mission_control_bg.xml
	AndroidLib/res/drawable-mdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-mdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable-mdpi/ic_loiter.png
	AndroidLib/res/drawable-port/mission_control_bg.xml
	AndroidLib/res/drawable-sw540dp/mission_control_bg.xml
	AndroidLib/res/drawable-xhdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-xhdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable-xhdpi/ic_loiter.png
	AndroidLib/res/drawable-xxhdpi/ic_action_quad_connected.png
	AndroidLib/res/drawable-xxhdpi/ic_action_quad_disconnect.png
	AndroidLib/res/drawable/flight_control_button_selector.xml
	AndroidLib/res/drawable/ic_action_quad.xml
…into feature_android_lib

Conflicts:
	Android/src/org/droidplanner/android/activities/EditorActivity.java
arthurbenemann added a commit that referenced this pull request Oct 14, 2014
@arthurbenemann arthurbenemann merged commit e65212a into DroidPlanner:master Oct 14, 2014
@m4gr3d m4gr3d deleted the feature_android_lib branch October 14, 2014 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants