Skip to content

Add Insight anonymously report usage statistics #744

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 2, 2015
Merged

Add Insight anonymously report usage statistics #744

merged 1 commit into from
Sep 2, 2015

Conversation

@zckrs
Copy link
Collaborator

@zckrs zckrs commented Aug 31, 2015

No description provided.

@zckrs zckrs force-pushed the feat-insight branch 3 times, most recently from ca82dcb to 3bd8b3a Compare August 31, 2015 14:42
@Swiip
Copy link
Owner

Swiip commented Sep 1, 2015

👍

Still WIP ?

@zckrs
Copy link
Collaborator Author

zckrs commented Sep 1, 2015

Still WIP. Actualy we just track the first run generator.

@zckrs
Copy link
Collaborator Author

zckrs commented Sep 1, 2015

Options:

  • track each prompt selection in different event ('/jquery2', '/restangular', '/ui.router', ...)
  • track user profil ('/jquery2/restangular/ui.router/...', '/zeptoJS/$http/new-router/...', ...)
  • or both

@zckrs
Copy link
Collaborator Author

zckrs commented Sep 2, 2015

Examples data send by Insight:

  • /jquery2/angular-resource/ui-router/bootstrap/noBootstrapComponents/node-sass/noJsPrepro/noHtmlPrepro
  • /jqLite/restangular/new-router/material-design-lite/null/less/typescript/jade

zckrs
zckrs reviewed Sep 2, 2015
View reviewed changes
app/src/prompts.js Outdated
Copy link
Collaborator Author

@zckrs zckrs Sep 2, 2015

Choose a reason for hiding this comment

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

Ugly line.
@Swiip should I use lodash to compose a object ?

Copy link
Owner

@Swiip Swiip Sep 2, 2015

Choose a reason for hiding this comment

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

Yep, perhaps it's better. lodash -> array -> this.insight.track.apply

Add yeoman/insight
3a4da52 
* 1cff17b Add Insight anonymously report usage statistics
* e715981 Add test on checkInsight
* b8a506c Send data by Insight
* 4b56a2e Replace prompts key none by explicit key
* fd0cb95 Fix tests
* 4e9a6d0 Compose array for insight.track
* e5bf0aa Complete test
@zckrs zckrs changed the title [WIP] Add Insight anonymously report usage statistics Add Insight anonymously report usage statistics Sep 2, 2015
Swiip added a commit that referenced this pull request Sep 2, 2015
@Swiip
Merge pull request #744 from Swiip/feat-insight
2128307 
Add Insight anonymously report usage statistics
@Swiip Swiip merged commit 2128307 into master Sep 2, 2015
@Swiip Swiip deleted the feat-insight branch September 9, 2015 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zckrs @Swiip