Page links on 7/30 Top Pages graphs are broken

[thekaveman]

Seems like they aren't including the protocol. Instead of

href="https://www.smgov.net/"

we have

href="www.smgov.net/"

making them relative to the analytics domain.

Now is working just fine.

[allejo]

I think this seems to be an issue with how Google gives us data for the 7/30 day reports. top-pages-7-days uses the 'ga:hostname' dimension which doesn't seem to include the protocol...? Before we just assumed HTTP so it conveniently worked out. The realtime report uses the hostname we define but the 7 & 30 days uses the hostname that Google gives us. Let me see what I can do to work around the issue

[allejo]

Quick fix deployed, the aggregate script will receive some much needed TLC in the feature/aggregate-script-46 branch so I'll incorporate a cleaner solution there.

[thekaveman]

That definitely fixed the issue, but... yikes 😨

Looking forward to the TLC. Let's keep this issue open as a reminder until then.