Skip to content

Update Walmart Case Study #1125

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

Merged
merged 2 commits into from
Feb 7, 2017
Merged

Update Walmart Case Study #1125

merged 2 commits into from
Feb 7, 2017

Conversation

ZibbyKeaton
Copy link
Contributor

Hey All - I'm trying to track downloads to our case studies. I'm doing a little test with Walmart to make sure it works. LMK if I did this correctly. cc: @phillipj

@phillipj
Copy link
Member

phillipj commented Feb 3, 2017

Please remove the [Read the full case study.] preceding the <a ...> you've added

@phillipj
Copy link
Member

phillipj commented Feb 3, 2017

Linking with #1121

@ZibbyKeaton
Copy link
Contributor Author

@phillipj I just did this. Let me know if this works for you.

@phillipj phillipj merged commit 68be475 into nodejs:master Feb 7, 2017
@phillipj
Copy link
Member

phillipj commented Feb 7, 2017

It is live and seems to be working. Hopefully you see these events in GA as well?

@ZibbyKeaton
Copy link
Contributor Author

@phillipj it is working. Thank you, thank you, thank you for helping me with this. You were an awesome help with this and really patient with me. I really appreciate it.

@phillipj
Copy link
Member

phillipj commented Feb 7, 2017

@ZibbyKeaton no problem! Feel free to open a new PRs if you want to do the same with more links.

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.

2 participants