Skip to content
This repository has been archived by the owner on Jul 19, 2021. It is now read-only.

add tracking for zip file #109

Merged
merged 2 commits into from
Mar 21, 2017
Merged

add tracking for zip file #109

merged 2 commits into from
Mar 21, 2017

Conversation

NathanPJF
Copy link
Contributor

Interested in seeing how often people are downloading the zip from the docs site as opposed to GitHub.

@NathanPJF NathanPJF requested review from bertiful and t-kelly March 20, 2017 00:41
@@ -20,4 +20,5 @@
</div>
</main>
</body>
<script src="{{ "/js/tracking.js" | prepend: site.baseurl }}"></script>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

async that bis-natch

@@ -10,7 +10,9 @@ Slate allows you to sync local files with your live shop, deploy to multiple env

## Getting started

> [Node](https://nodejs.org/en/) 4+ is supported, but 6+ is recommended to fully benefit from Slate. If you want the template files without the build tools, [get the latest zip here](https://sdks.shopifycdn.com/slate/latest/slate-src.zip).
<blockquote>
<a href="https://nodejs.org/en/">Node</a> 4+ is supported, but 6+ is recommended to fully benefit from Slate. If you want the template files without the build tools, <a href="https://sdks.shopifycdn.com/slate/latest/slate-src.zip" data-ga-track="slateZip">get the latest zip here</a>.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity, do you remember why we chose Node 4+ to be the minimum requirement?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure, maybe @macdonaldr93 could tell us.

@NathanPJF NathanPJF merged commit d7b02d5 into master Mar 21, 2017
@NathanPJF NathanPJF deleted the zip-tracking branch March 21, 2017 04:16
@lock
Copy link

lock bot commented Oct 26, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants