search

opengd / solar-sis

solar serial inverter system
MIT License
33 stars 15 forks source link

Latest npm not updated #16

Closed cybertza closed 3 years ago

cybertza commented 4 years ago

https://www.npmjs.com/package/solar-sis does not reflect the latest git data for index.js

I had to do a reinstall, i managed to pull the git, and the git and the data in the index.js reflects the latest fix to allow you to send strings to InFluxDb, but when you do a NPM install, it seems to use the outdated index.js from there for the folder solar-sis/example/../node_modules/solar-sis/index.js

end result is the fix for being able to parse string from calls is not there, and then InFluxDb disregards the send.

Since the Error Handling is not displaying in the log, it's not easy to track down.

cybertza commented 4 years ago

updated the project to better show db save issues and mqtt support

18

cybertza commented 4 years ago

i also found this, making the NPM not as important...

https://github.com/opengd/solar-sis/commit/c39a8a30d7acfcf7d63d57afc086ff4ecadcdf6e

var mpi = require("../../index");

should prob update all of the files to work like this in the future.