Deploy Potatolytics (packaged app-patible GA)

RESOLVED FIXED in 2014-04-29

Status

Marketplace
Consumer Pages
P1
normal
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: basta, Assigned: garethc)

Tracking

(Blocks: 1 bug)

2014-04-29
x86_64
Windows 7
Points:
---
Dependency tree / graph

Details

(Reporter)

Description

5 years ago
GA isn't compatible with a packaged app, so we should be working hard to create a solution based on the feedback of the GA premium support team:

1. Create an iframe which embeds GA
2. Use postMessage to pass page views, events, and variables to the iframe

Pretty straightforward, but we need to test this and make sure it doesn't gum anything up, namely getting weird URLs due to passing a full URL (with origin) to the GA page view method.

Comment 1

5 years ago
This will be awesome to do right and we should openly share how we accomplished it.

Here is some code examples to help us:

https://github.com/GoogleChrome/chrome-app-samples/tree/master/analytics

I also read people using this:

https://github.com/ggendre/GALocalStorage
(Reporter)

Updated

5 years ago
Blocks: 897156
(Reporter)

Comment 3

5 years ago
Testing has started for the second time today. We'll see if there are any more issues that need to be worked out.
(Reporter)

Comment 4

5 years ago
This is unexpectedly blocked on bug 897146. Inferno is acting up, which means getting the ball rolling with Universal Analytics might take until next week.
Depends on: 897146
Assignee: nobody → mattbasta
"Hopefully we will be able to deploy potatolytics next week"
Priority: -- → P3

Updated

5 years ago
Blocks: 903132
(Reporter)

Comment 6

5 years ago
UA landed on -dev today:

https://github.com/mozilla/fireplace/commit/a4f72bb608b9a6c4b20ab76e0438bca52f6a97c6

I'll start on the potato side of it tomorrow. It should actually be pretty easy; UA lets you disable storage entirely and instead provide your own client ID. We've already got that part in place, so the only challenge is going to be ripping out the GA-potato code and replacing it with UA-potato code.
(Reporter)

Comment 7

5 years ago
This adds potatolytics support to UA:

https://github.com/mozilla/fireplace/commit/5b97ae6a6981ea57e4d36d5e66238cb642844640

I'll start testing it locally and maybe flip it on briefly on -dev. Still waiting on gcull to give the thumbs up on the UA data, so I don't want to completely switch it over if there are bugs with the underlying UA abstraction layer.
(Reporter)

Comment 8

5 years ago
This is built, assigning to Gareth since we're blocked on his analysis of the current UA implementation.

Gareth, any update?
Assignee: mattbasta → garethcull.bugs
Summary: Create Potatolytics (packaged app-patible GA) → Deploy Potatolytics (packaged app-patible GA)
(Assignee)

Comment 9

5 years ago
I'll review and will get back to you with an updated analysis later this week. 

Gareth
(In reply to Gareth Cull [:garethc] from comment #9)
> I'll review and will get back to you with an updated analysis later this
> week. 
> 
> Gareth

How'd it go?
(Assignee)

Comment 11

5 years ago
Hi Wil,

At a high level, the aggregate data looks pretty comparable across a variety of metrics. But, I am seeing some differences when I break it down by some segments. I'm reviewing with Matt tomorrow if you would like to join. Also, I'm not sure if you know this, but Google recently announced that we will be able to migrate our existing Classic Implementation to Universal Analytics with an upgrade tool. This would be great, as we wouldn't lose any historical data when switching over to a UA only account. Something to discuss. 

More on this here: http://analytics.blogspot.ca/2013/10/an-easy-way-to-upgrade-to-universal_23.html

Gareth
Hi Gareth - we're looking into revisiting this shortly.  Can you give an update on your last meeting and next steps here?
Flags: needinfo?(garethcull.bugs)
(Assignee)

Comment 13

4 years ago
Hey Wil,

Universal Analytics (UA) is out of beta now, and we can migrate our current Classic GA implementation over to UA. Switching over to UA will require us to update any custom variables that we are passing on marketplace in addition to updating the GA snippet. I believe some of your team maybe using some custom variables in your reporting, which would necessitate the need to update.

Let me put together an etherpad outlining some of the changes we will need to make. Could you please schedule a quick meeting with me to review sometime next week? Thanks.
Flags: needinfo?(garethcull.bugs)
Blocks: 998811
Priority: P3 → P1
Target Milestone: --- → 2014-04-29

Updated

4 years ago
Blocks: 1001040

Comment 14

4 years ago
functionality can be flipped with potatolytics_enabled flag.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Comment 15

4 years ago
Please add STR here or mark it with [qa-] if no QA is needed.

Updated

4 years ago
Flags: needinfo?(mattbasta)
(Reporter)

Comment 16

4 years ago
Iirc this was enabled on dev last year. If not, the qa process involves turning it on for dev and comparing incoming data with historical data.
Flags: needinfo?(mattbasta)
You need to log in before you can comment on or make changes to this bug.