Decouple the data source population code to a separate repo.

Author: zachleat

.gitignore

@@ -19,6 +19,3 @@ dist
 
 # Environment variables
 .env
-
-# Captions Metadata from Jamstack TV
-data/

algolia.config.js

@@ -3,7 +3,6 @@ require('dotenv').config();
 module.exports = {
   appId: 'VGOL4P01VV',
   indexName: 'jamstack',
-  apiKey: process.env.ALGOLIA_API_KEY,
   // It’s okay to have this in the repo, it’s used in the clientside JS
   searchOnlyApiKey: "a457b566acbf454a61eaaae2e4fee0bf",
 };

netlify.toml

@@ -6,18 +6,6 @@
   NODE_VERSION = "12.16.2"
   NODE_ENV = "production"
 
-# This requires YOUTUBE_API_KEY and ALGOLIA_API_KEY
-[context.production]
-  command = "npm run build:production"
-
-# TODO remove this, this is just for testing
-[context.deploy-preview]
-  command = "npm run build:production"
-
-# TODO remove this, this is just for testing
-[context.branch-deploy]
-  command = "npm run build:production"
-
 [dev]
   command = "npm run start"
   publish = "dist"

package.json

@@ -15,13 +15,11 @@
     "build": "npm-run-all build:html build:css",
     "build:html": "eleventy",
     "build:css": "postcss src/css/tailwind.css -o dist/css/styles.css",
-    "data:jamstacktv": "youtinx pull && youtinx push",
     "watch:html": "ELEVENTY_ENV=dev eleventy --watch --quiet",
     "watch:css": "postcss src/css/tailwind.css -o dist/css/styles.css --watch",
     "serve": "live-server dist --quiet --port=8090",
     "start": "npm-run-all --parallel watch:html watch:css serve",
-    "clean": "rm -rf dist",
-    "build:production": "npm run data:jamstacktv && npm run build"
+    "clean": "rm -rf dist"
   },
   "dependencies": {
     "@11ty/eleventy": "^0.11.1",
@@ -47,8 +45,7 @@
     "postcss-cli": "^8.3.1",
     "postcss-import": "^13.0.0",
     "spdx-correct": "^3.1.1",
-    "tailwindcss": "^2.0.3",
-    "youtinx": "^0.4.1"
+    "tailwindcss": "^2.0.3"
   },
   "devDependencies": {
     "live-server": "^1.2.1"

src/site/_data/algolia.js

@@ -1,6 +1,6 @@
 const algoliaConfig = require("../../../algolia.config.js");
 
-// make sure we don’t leak algoliaConfig.apiKey
+// don’t hardcode anything here, make the config variables opt-in
 module.exports = {
   appId: algoliaConfig.appId,
   indexName: algoliaConfig.indexName,

src/site/tv.njk

@@ -9,9 +9,10 @@ layout: layouts/base.njk
   <div id="searchbox"></div>
   <div id="hits"></div>
 </div>
-<script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/algoliasearch-lite.umd.js" integrity="sha256-EXPXz4W6pQgfYY3yTpnDa3OH8/EPn16ciVsPQ/ypsjk=" crossorigin="anonymous"></script>
-<script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/instantsearch.production.min.js" integrity="sha256-LAGhRRdtVoD6RLo2qDQsU2mp+XVSciKRC8XPOBWmofM=" crossorigin="anonymous"></script>
-<script>
+
+<script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/algoliasearch-lite.umd.js" integrity="sha256-EXPXz4W6pQgfYY3yTpnDa3OH8/EPn16ciVsPQ/ypsjk=" crossorigin="anonymous" defer></script>
+<script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/instantsearch.production.min.js" integrity="sha256-LAGhRRdtVoD6RLo2qDQsU2mp+XVSciKRC8XPOBWmofM=" crossorigin="anonymous" defer></script>
+<script defer>
 const searchClient = algoliasearch('{{ algolia.appId }}', '{{ algolia.searchOnlyApiKey }}');
 
 const search = instantsearch({