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

Adds optimizely snippet to head #1121

Merged
merged 1 commit into from
Aug 3, 2016
Merged

Conversation

ryankeairns
Copy link
Contributor

Going to test some copy changes.

Note:
A previous attempt at adding Optimizely through Google Tag Manager caused issues since that lodges the script into the body. I've added it directly into the head here (not using GTM) and everything looks 👌 locally.

@thesentinels
Copy link
Contributor

@ryankeairns, thanks for your PR! By analyzing the annotation information on this pull request, we identified @cwebberOps, @TrevorBramble and @magwalk to be potential reviewers

Signed-off-by: Ryan Keairns <[email protected]>
@ryankeairns ryankeairns force-pushed the rk/www-optimizely-snippet branch from 13b0e75 to 6f8f19c Compare August 3, 2016 20:52
@reset
Copy link
Collaborator

reset commented Aug 3, 2016

gif-keyboard-14100617156946726633

@thesentinels r+

@thesentinels
Copy link
Contributor

📌 Commit 6f8f19c has been approved by reset

@thesentinels
Copy link
Contributor

⌛ Testing commit 6f8f19c with merge 35ebdac...

thesentinels pushed a commit that referenced this pull request Aug 3, 2016
Signed-off-by: Ryan Keairns <[email protected]>

Pull request: #1121
Approved by: reset
@thesentinels
Copy link
Contributor

☀️ Test successful - travis

@thesentinels thesentinels merged commit 6f8f19c into master Aug 3, 2016
@smurawski smurawski deleted the rk/www-optimizely-snippet branch February 3, 2017 17:32
@eeyun eeyun mentioned this pull request Jan 25, 2018
10 tasks
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.

4 participants