Skip to content

Added a special param to measure activity on amp-hulu#7322

Merged
aghassemi merged 1 commit intoampproject:masterfrom
hulu:amp-hulu
Feb 12, 2017
Merged

Added a special param to measure activity on amp-hulu#7322
aghassemi merged 1 commit intoampproject:masterfrom
hulu:amp-hulu

Conversation

@unbug
Copy link
Copy Markdown
Contributor

@unbug unbug commented Feb 3, 2017

For now we don't have a endpoint to track traffic from amp-hulu, add a param to the embed url helps embed player and hulu server to identify the request.

  1. Add "amp=1" to the umbed url.
    https://secure.hulu.com/dash/mobile_embed.html?amp=1

@jridgewell
Copy link
Copy Markdown
Contributor

/to @aghassemi

@aghassemi aghassemi merged commit 965598f into ampproject:master Feb 12, 2017
torch2424 pushed a commit to torch2424/amphtml that referenced this pull request Feb 14, 2017
@unbug unbug deleted the amp-hulu branch April 12, 2017 11:10
mrjoro pushed a commit to mrjoro/amphtml that referenced this pull request Apr 28, 2017
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.

3 participants