updates to implementation.html

I think there are two issues that need to be addressed, and we can reword the warning. 1) People need to understand they cannot just copy and paste the example code - they need to actually replace the filler values with actual values from their CMS. 2) Call out which parameters are actually required and which ones are optional (language like ‘chosen parameters’ makes them all sounds optional'

On November 12, 2014 at 10:57:14 AM, Liz Baillie (notifications@github.com) wrote:

In source/implementation.html.erb:

> @@ -52,33 +51,120 @@ __reach_config = {
>    (document.getElementsByTagName('head')[0] || document.getElementsByTagName('body')[0]).appendChild(s);
>  })();
>    <% end %>
> +  </div>
>   
I re-worded the instructions directly above the code to be more explicit about how the code should be used, basically saying what was in the warning. Also replaced the date with $publication_date to make it more obvious that they shouldn't just copy/paste the date that was there. But if you think having the warning below the code in addition to that would be helpful, I see no harm in putting it back there.

—
Reply to this email directly or view it on GitHub.