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

Google Analytics update #829

Merged
merged 19 commits into from
Jun 28, 2014
Merged

Google Analytics update #829

merged 19 commits into from
Jun 28, 2014

Conversation

m4gr3d
Copy link
Member

@m4gr3d m4gr3d commented Jun 27, 2014

Added analytic reporting as discussed:

  • active droneshare account
  • Flight action buttons usage
  • Mavlink connection mechanism

In the process, the GA reporting logic was refactored into its own class.

Updated the gradle build file to support android-studio beta, and gradle 1.12

m4gr3d added 19 commits June 12, 2014 15:24
added preference to disable usage statistics reporting.
added 'up' navigation button to the settings screen.
added a Category enum to store all analytics related categories.
…into feature_analytics

Conflicts:
	Android/res/xml/preferences.xml
…into feature_analytics

Conflicts:
	Android/res/xml/preferences.xml
fixed license issue with travis ci.
updated connection type session on disconnection.
@arthurbenemann
Copy link
Member

I'm merging this, even if the commit message on 354d681 makes me a little nervous.

But the C.I. server makes me much more confident to make the merge blindly (I'm not testing this individually since it's a change that doesn't affect flight code).
https://travis-ci.org/DroidPlanner/droidplanner/builds/28612777

arthurbenemann added a commit that referenced this pull request Jun 28, 2014
@arthurbenemann arthurbenemann merged commit 61b63a3 into master Jun 28, 2014
@arthurbenemann arthurbenemann deleted the feature_analytics branch June 28, 2014 05:05
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.

2 participants