NREL / mobility_landscape

Apache License 2.0
1 stars 3 forks source link

Inability to remove twitter, ads, and cards #11

Open jpegburch opened 3 years ago

jpegburch commented 3 years ago

These current options within settings.yml are backed up with multiple steps to produce them within the landscape. For example, when removing the twitter options within settings.yml, the file tweetbutton.js files a failure at text: settings.twitter.text, and cannot run. When attempting to simply just remove this one line, the code fails in a different spot, and similarly fails when trying to just remove the whole file. A similar story follows for the other options, suggesting that three options are in the code in multiple places. It might be easier to just add our own version of each instead of removing each of them, but open to suggestion.

ljanicke commented 3 years ago

Changed the CNCF ad to be a NREL logo linked to the NREL website.

shankari commented 3 years ago
Screen Shot 2021-07-15 at 2 43 55 PM
shankari commented 3 years ago

Removed the ad; not sure about twitter et al