use sane-topojson files from node_modules not vendor

Author: archmoj

devtools/test_dashboard/devtools.js

@@ -5,6 +5,7 @@
 var Fuse = require('fuse.js/dist/fuse.common.js');
 var mocks = require('../../build/test_dashboard_mocks.json');
 var credentials = require('../../build/credentials.json');
+var constants = require('@src/plots/geo/constants');
 var Lib = require('@src/lib');
 var d3 = require('../../test/strict-d3');
 var d3Json = d3.json;
@@ -19,7 +20,7 @@ var Tabs = {
         Plotly.setPlotConfig({
 
             // use local topojson files
-            topojsonURL: '../../vendor/topojson/',
+            topojsonURL: constants.pathToTopojsonSrc,
 
             // register mapbox access token
             // run `npm run preset` if you haven't yet

package.json

@@ -21,12 +21,12 @@
     "plotly"
   ],
   "scripts": {
-    "preprocess": "node tasks/preprocess.js",
     "bundle": "node tasks/bundle.js",
     "header-dist": "node tasks/header_dist.js",
     "stats": "node tasks/stats.js",
     "find-strings": "node tasks/find_locale_strings.js",
-    "build": "node tasks/empty_dist.js && npm run preprocess && node tasks/topojson_dist.js && npm run find-strings && npm run bundle && npm run header-dist && npm run stats",
+    "preprocess": "node tasks/preprocess.js",
+    "build": "node tasks/empty_dist.js && npm run preprocess && npm run find-strings && npm run bundle && npm run header-dist && npm run stats",
     "cibuild": "node tasks/empty_dist.js && npm run preprocess && node tasks/cibundle.js",
     "watch": "node tasks/watch.js",
     "lint": "eslint --version && eslint .",

tasks/preprocess.js

@@ -24,11 +24,11 @@ function makeBuildCSS() {
     });
 }
 
-// copy topojson files from sane-topojson to vendor/
+// copy topojson files from sane-topojson to dist/
 function copyTopojsonFiles() {
     fs.copy(
         constants.pathToTopojsonSrc,
-        constants.pathToTopojsonVendor,
+        constants.pathToTopojsonDist,
         { clobber: true },
         common.throwOnError
     );

tasks/topojson_dist.js

@@ -65,7 +65,6 @@ module.exports = {
 
     pathToTopojsonSrc: pathToTopojsonSrc,
     pathToTopojsonDist: path.join(pathToDist, 'topojson/'),
-    pathToTopojsonVendor: path.join(pathToVendor, 'topojson/'),
     pathToPlotlyGeoAssetsSrc: path.join(pathToSrc, 'assets/geo_assets.js'),
     pathToPlotlyGeoAssetsDist: path.join(pathToDist, 'plotly-geo-assets.js'),
 

tasks/util/constants.js

@@ -24,7 +24,7 @@ var HOVERMINTIME = require('@src/components/fx').constants.HOVERMINTIME;
 
 // use local topojson files
 Plotly.setPlotConfig({
-    topojsonURL: '/base/vendor/topojson/'
+    topojsonURL: constants.pathToTopojsonSrc
 });
 
 function move(fromX, fromY, toX, toY, delay) {