jewelbots / www Goto Github PK

When following the instructions in the README I get an error that the favicon isn't there. On examination there is no /dist folder.

Right now we have one instance during deployments; if a deploy 'goes bad', we have nothing to fall back to. Dependent upon #28 .

Right now this is a static site driven off of Nginx. Changing it over to node.js will allow us to implement Mailchimp's API, as well as clean up a number of issues, including having to duplicate header and footer scripts across pages.

The necessary files to deploy Jewelbots are kept in a few separate Repos; leading to a manual building of the files necessary to deploying the site. This is sub-optimal.

Currently, if a developer makes changes to the nginx file, those changes are shown on the same file we use for production; should have a dist/ folder that includes distribution files (whether for local dev or production) so standard files are not modified.

The Bootstrap script (bootstrap.sh) should be able to be run multiple times without adverse side effects. As pointed out in the README.md in #46, running it twice results in multiple KVPs being insert into the same keys/mc.json file.

Here's the offending lines:

touch keys/mc.json
echo -e "{\n\t\"apiKey\":\"\"\n}" >> ./keys/mc.json

Proposed fix would be to add an if block around the echo and ensure that file is empty.