MMM-PlaceInfo: display information about the world

Dresch360

Unable to get modules to load. Is this still being supported?

njw

@dresch360 It is still supported. I wasn’t watching the forums for questions, sorry. Just “woke up” 😀. Pushed a new release that has more debug info in case it helps. Let me know what errors you get or what config you’re using if it’s not working for you.

Dresch360

@njw Many thanks! It works with your new update.

raccoonpie

@njw it seems that something changed since last time. The currency API key is not working. The Base URL in the JS file needs to be updated line 57 to

currencyAPI: "https://api.apilayer.com/fixer/latest",

A small tweak in the line 458 to :

params = "?apikey=" + this.config.currencyAPIKey;

Could you please check and if possible provide an update? Thanks for your contribution, it is a beautiful and helpful module. Best!

sdetweil

@raccoonpie fork the repo

download your fork
make the changes
do a git add for the changed files
and then a git commit -m “…whatever message to describe the changes”
then a git push to update the github copy of your fork

then create a pull request of your changes back to the original.

this way, you have updated the code and the author can accept your pull request and update the master without having to write the code himself…

welcome to open source community development

raccoonpie

@sdetweil I will, thank you!