Skip to content

Add Google Analytics #22

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

Merged
merged 1 commit into from
Jul 29, 2016
Merged

Add Google Analytics #22

merged 1 commit into from
Jul 29, 2016

Conversation

matklad
Copy link
Member

@matklad matklad commented Jul 29, 2016

closes #20

@mkaput
Copy link
Member

mkaput commented Jul 29, 2016

I'd like to emphasise my worries about this.

I agree that some analytics would be nice; though I'd prefer them to be publicly available; not only visible to the person who has set them up.

As an open source project, I believe limiting visibility of some data related to it is slightly inappropriate. I agree that stuff like IP addresses must not be public, but they can be easily anonymised.

Even personally, as a person who authored this website, I would just love to see such stats, but not with the cost of appropriating this privilege.

I think, different analytics provider is the way to go here. Open solutions like Piwik seems reasonable, but somebody has to set up them. I have no funds to do this unfortunately.

@matklad
Copy link
Member Author

matklad commented Jul 29, 2016

I also would prefer an open solution, but in the free and "as a service" form. I don't want to put an effort into maintaining our own analytics at this stage of the project :)

I am also Ok with using GA (and https://www.rust-lang.org/ does this anyway ;) ) Having this info will probably make this web site better, but I don't think it is critical.

Feel free to reopen if you change your opinion about GA or find a more open and out of the box solution.

@matklad matklad closed this Jul 29, 2016
@mkaput mkaput reopened this Jul 29, 2016
@mkaput
Copy link
Member

mkaput commented Jul 29, 2016

lgtm as for now, and replace this with open alternative when we find out how

@mkaput mkaput merged commit a38637b into master Jul 29, 2016
@mkaput mkaput deleted the ga branch August 5, 2016 18:33
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.

Add google analytics?
2 participants