FORGE-224: Added Tracking APIs #208

Merged
merged 1 commit into from Jan 7, 2013

Conversation

Projects
None yet
2 participants
@gastaldi
Member

gastaldi commented Sep 11, 2012

Added Google Analytics support. Don't push this until reviewed

@ghost ghost assigned lincolnthree and gastaldi Sep 11, 2012

@lincolnthree

View changes

tracking/src/main/java/org/jboss/forge/tracking/GoogleAnalyticsPlugin.java
+ {
+ enabled = shell
+ .promptBoolean(
+ "Will you allow Forge team to receive anonymous usage statistics for this Forge installation ?",

This comment has been minimized.

@lincolnthree

lincolnthree Sep 19, 2012

Member

"Will you allow the Forge team to receive anonymous usage statistics for this instance of JBoss Forge?"

@lincolnthree

lincolnthree Sep 19, 2012

Member

"Will you allow the Forge team to receive anonymous usage statistics for this instance of JBoss Forge?"

+ * Display info about the analytics usage
+ */
+ @Command("info")
+ public void displayInfo()

This comment has been minimized.

@lincolnthree

lincolnthree Sep 19, 2012

Member

This needs to print out all of the information that is being sent, like this: http://www.jboss.org/dms/tools/images/usage-reporting-preferences.png - this can just be printed to the console as a key-value pair.


Forge Version: 1.1.0.Final
Operating system: Windows
Java Version: 1.6.0_32_b12
Plugins installed: X, Y, Z

@lincolnthree

lincolnthree Sep 19, 2012

Member

This needs to print out all of the information that is being sent, like this: http://www.jboss.org/dms/tools/images/usage-reporting-preferences.png - this can just be printed to the console as a key-value pair.


Forge Version: 1.1.0.Final
Operating system: Windows
Java Version: 1.6.0_32_b12
Plugins installed: X, Y, Z

This comment has been minimized.

@gastaldi

gastaldi Dec 11, 2012

Member

Done. Now I just need to fit all this info in the Analytics API. Thanks !

@gastaldi

gastaldi Dec 11, 2012

Member

Done. Now I just need to fit all this info in the Analytics API. Thanks !

@gastaldi

This comment has been minimized.

Show comment
Hide comment
@gastaldi

gastaldi Jan 7, 2013

Member

It should be ok now. @lincolnthree: could you review it again ? Thanks

Member

gastaldi commented Jan 7, 2013

It should be ok now. @lincolnthree: could you review it again ? Thanks

gastaldi added a commit that referenced this pull request Jan 7, 2013

Merge pull request #208 from gastaldi/tracking
FORGE-224: Added Tracking APIs

@gastaldi gastaldi merged commit b713a9b into forge:master Jan 7, 2013

@gastaldi gastaldi deleted the gastaldi:tracking branch Jan 7, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment