search

asantaga / lightwaverf_HA_EnergySensor

Home Assistant Sensor for the LightwaveRF energy monitor
MIT License
6 stars 3 forks source link

hassio log errors - manifest.json #1

Closed MJP-76 closed 5 years ago

MJP-76 commented 5 years ago

Hi There,

Firtly, Great work, and realy appreciate you putting this together

I have found an issue and resolved the it, hope it helps unless its my setup. So, I am ruinning latest HASSIO and just getting back to test this component and getting the following error in my log

"Error parsing manifest.json file at /config/custom_components/lightwave_energy/manifest.json: Expecting ',' delimiter: line 5 column 3 (char 162)"

In the manifest.json at line 4 it has "documentation": "https://www.home-assistant.io/components/lightwave_energy"

It should however be "documentation": "https://www.home-assistant.io/components/lightwave_energy",

When I fix as above, my homeassiosant then wont start and gets tonnes of python errors so removed the custom component and hassio started fine

I made various changes and the following is the result in my manifest.json and my HASSIO starts up fine and creates the sensors as expected etc

{ "domain": "lightwave_energy", "name": "Lightwave Energy Monitor Platform", "documentation": "https://www.home-assistant.io/components/lightwave_energy", "codeowners": [ "@asantaga" ], "requirements": [ ], "dependencies": [ ] }

Cheers Matthew

asantaga commented 5 years ago

Hey Matthew, Not sure what was wrong with the manifest file but Ive fixed it now.. I actually recreated it from scratch and it now works fine..

can u pull the latest commit and let me know if its all ok?

if it is then we can close this ticket

MJP-76 commented 5 years ago

Will Pull and try again and update

If i have any problems, will open again

Cheers

MJP-76 commented 5 years ago

Working like a charm now, Cheers mate

asantaga commented 5 years ago

awesome!

On Fri, 21 Jun 2019 at 14:06, matthewjporter notifications@github.com wrote:

Working like a charm now, Cheers mate

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/asantaga/lightwaverf_HA_EnergySensor/issues/1?email_source=notifications&email_token=AAWN2QDKLGGY6AJOEHFVQE3P3TG6ZA5CNFSM4HYNKMV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYINCTA#issuecomment-504418636, or mute the thread https://github.com/notifications/unsubscribe-auth/AAWN2QDDLWWKZLIP5I6JIZLP3TG6ZANCNFSM4HYNKMVQ .

--

Angelo