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

Piwik tracking for CNAME Akvo Pages not working after upgrade to V3.0 #1305

Closed
Lynngre opened this issue Mar 5, 2015 · 9 comments
Closed
Assignees

Comments

@Lynngre
Copy link

Lynngre commented Mar 5, 2015

There is currently no tracking happening for the partner Akvo Pages ( CNAMES)

Last stats were from the Thursday of the upgrade. Was the JS code included in the upgraded pages?
#630

@Lynngre
Copy link
Author

Lynngre commented Mar 6, 2015

@adriancollier not sure when this could be fixed, the data is not recoverable, so for as long as it's not being collected there will be no data - not sure how bad that is for partners (which partners were notified of this being available). Either way there will be a hole in what we can provide.

@adriancollier
Copy link
Contributor

We should add this back into the code. If I remember correctly, it's not a big fix and should be easy enough for us to fix it before the next release.
@KasperBrandt can you take a look?

@KasperBrandt
Copy link
Contributor

It seems the code hasn't changed from v2, so I'm not sure what's causing this.

@Lynngre, do you get tracking from regular Akvo Pages? It seems like we fill in site id=1, regardless of core RSR or an Akvo Page, everywhere..

@Lynngre
Copy link
Author

Lynngre commented Mar 12, 2015

Akvoapp.org is no longer tracked as part of akvo.org or rsr.akvo.org as per the issue #630

So any *.akvoapp.org or CNAME for those should be using their allocated SITEID as per list here:

https://docs.google.com/spreadsheets/d/1pfwOmf4aMWx4YEwYqD7SFYkc7-Z3hApcTVTQgZ5lARY/edit?usp=sharing

@adriancollier
Copy link
Contributor

All of the IDs are present within the data at rsr_partnersite.piwik_id so we just need to add the code to put this into the right templates.

@Lynngre
Copy link
Author

Lynngre commented Mar 12, 2015

Great, thanks guys!

@Lynngre
Copy link
Author

Lynngre commented Apr 8, 2015

So this was removed from the last update? We seriously need this to be done asap.

@kardan
Copy link
Contributor

kardan commented Apr 8, 2015

@Lynngre it's getting into the next release, thanks for the bump!

kardan added a commit that referenced this issue Apr 9, 2015
@KasperBrandt KasperBrandt reopened this Apr 9, 2015
kardan added a commit that referenced this issue Apr 9, 2015
- Added correct piwik template tag
- Added simple tests for widgets
- Small syntax cleanup
@KasperBrandt
Copy link
Contributor

Test plan

GIVEN Akvo Pages (also with CNAME)
WHEN a Piwik code is filled in for the Akvo Page
THEN this code should be filled in at the Piwik tracking part

@MichaelAkvo MichaelAkvo moved this to Done in RSR Dec 8, 2022
@MichaelAkvo MichaelAkvo added this to RSR Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

4 participants