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.

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

PoweredBy and utmz #711

Closed
vvo opened this issue Dec 2, 2015 · 10 comments
Closed

PoweredBy and utmz #711

vvo opened this issue Dec 2, 2015 · 10 comments
Milestone
1.2

Comments

@vvo
Copy link
Contributor

vvo commented Dec 2, 2015

To be able to easily track where the users are coming from when clicking PoweredBy link, we should:
@redox
Copy link
Contributor

redox commented Dec 2, 2015

👍

@Jerska
Copy link
Member

Jerska commented Dec 8, 2015

Could we do it through a redirect as we're doing with https://www.algolia.com/console ? I'd be really less enclined to click a link where I see utm parameters.

@vvo
Copy link
Contributor Author

vvo commented Dec 8, 2015

Could we do it through a redirect as we're doing with https://www.algolia.com/console ?

It would require us to create redirects for each poweredBy implem, why won't you be good with clicking an utm link?

@Jerska
Copy link
Member

Jerska commented Dec 8, 2015

No, it just requires us to create a route that will check the referer.

why won't you be good with clicking an utm link?

The fact that I'll be the product of a company I don't know yet (or I wouldn't click the link).
Furthermore, as a webmaster, I'd be more willing to accept to have a link to another website on my website without marketing content in it.

@vvo
Copy link
Contributor Author

vvo commented Dec 8, 2015

The fact that I'll be the product of a company I don't know yet (or I wouldn't click the link).

Yes but you will be in the end even with a redirect, so hiding it might make it look worse?

Furthermore, as a webmaster, I'd be more willing to accept to have a link to another website on my website without marketing content in it.

I am not sure this has been an issue with our previous poweredBy implems, you got free search, you accept a logo, then a link yes that's the "contract" we can force people to without any friction I think

@Jerska
Copy link
Member

Jerska commented Dec 8, 2015

Yes but you will be in the end even with a redirect, so hiding it might make it look worse?

I agree on this. I remember it didn't shock me when I first clicked http://algolia.com/console (although I do remember it).
About the contract, why not.

@pixelastic
Copy link
Contributor

I agree with @jerskouille. I always check the url of the logo links I click. If I see some marketing elements in it, I might avoid clicking it, or copy-pasting the like, remove the tracking and going on.

I agree that in the end this is the exact same thing, but I'd rather we go the /console (or any other route) way, and funnel the analytics through the referer. If we can do it back-end, on our side, this is better than doing it client-side.

@vvo
Copy link
Contributor Author

vvo commented Dec 11, 2015

Really not seeing this as an issue, we should do the simplest thing for now by being transparent to everyone and not hiding the fact that yes we are tracking people: utmz etc..

@pixelastic
Copy link
Contributor

I'm also thinking about that :)
image

@vvo vvo added this to the 1.2 milestone Jan 8, 2016
@vvo vvo added the feature label Jan 8, 2016
@vvo
Copy link
Contributor Author

vvo commented Feb 1, 2016

implemented the default poweredBy link computation, not passing options. Not worth it

@vvo vvo closed this as completed in 05d1425 Feb 2, 2016
vvo pushed a commit that referenced this issue Feb 2, 2016
v1.2.0
3cbc294 
<a name="1.2.0"></a>
# [1.2.0](v1.1.3...v1.2.0) (2016-02-02)

### Bug Fixes

* **all:** typos ([fa8ba09](fa8ba09))
* **currentRefinedValues:** allow array of strings for cssClasses.* ([55b3a3f](55b3a3f))
* **docs:** fixed bad link to scss in custom themes section ([823a859](823a859))
* **getRefinements:** a name should be a string ([7efd1fd](7efd1fd))
* **getRefinements:** hierarchical facets ([fe0fc5d](fe0fc5d))
* **index:** Use module.exports instead of export on index ([81e7eee](81e7eee))
* **pagination:** remove default value of maxPages. Fixes #761 ([607fe9a](607fe9a)), closes [#761](#761)
* **prepareTemplates:** uses templates with keys that are not in defaults ([c4bf8ec](c4bf8ec))
* **rangeSlider:**     prevent slider from extending farther than the last pip ([6e534f5](6e534f5))
* **search-box:** update value when state changes from the outside ([4550f99](4550f99))
* **url-sync:** adds indexName in the helper configuration ([e50bafd](e50bafd))
* **url-sync:** Makes url sync more reliable ([3157abc](3157abc)), closes [#730](#730) [#729](#729)

### Features

* **currentRefinedValues:** new widget ([6c926d0](6c926d0)), closes [#404](#404)
* **hits:** adds allItems template as an alternative to item ([1f3f889](1f3f889))
* **poweredBy:** automatically add utm link to poweredBy ([05d1425](05d1425)), closes [#711](#711)
* **priceRanges:** add currency option ([f41484a](f41484a))
* **refinementlist:** lets configure showmore feature ([3b8688a](3b8688a))
* **Template:** accepts any parameters and forwards them ([5170f53](5170f53))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
5 participants
@redox @vvo @pixelastic @bobylito @Jerska