nprapps / dailygraphics-templates

Graphic templates for the dailygraphics-next rig
9 stars 14 forks source link

analytics script included twice on some templates #31

Closed DanielJWood closed 1 year ago

DanielJWood commented 1 year ago

It appears that the analytics script is included both in the pym.js script (where it should be) and in the graphic.js (where it used to be) in some of the templates.

For instance in the d3_graphic template, it's included at the top of graphic.js:

var pym = require("./lib/pym");
var ANALYTICS = require("./lib/analytics");
require("./lib/webfonts");
var { isMobile } = require("./lib/breakpoints");

It should be removed from graphic.js, we think, though I'd like to test to make sure it's still there after removal.