homeasssitant-schlage icon indicating copy to clipboard operation
homeasssitant-schlage copied to clipboard

manifest file typo

Open Admiral-99 opened this issue 2 years ago • 4 comments

Hello,

I could not figure out why the addon was not loading. Upon checking the logs I found this 2023-01-09 21:27:45.574 ERROR (SyncWorker_1) [homeassistant.loader] Error parsing manifest.json file at /config/custom_components/schlage/manifest.json: trailing comma is not allowed: line 11 column 2 (char 359)

Checking the manifest file, there is an additional comma on line 10 at the end that should not be there.

Thanks!

Admiral-99 avatar Jan 10 '23 03:01 Admiral-99

Has this been fixed. Still getting an error.

Screenshot_20230114_100946_Home Assistant.jpg

schergr avatar Jan 14 '23 15:01 schergr

+1 for typo on line 10 of the manifest.json, this prevents HACS from importing it and the integration loading properly. However, if you want you can still install the integration manually by adding the schlage folder of the repo to your custom components folder and manually editing out the comma on line 10 of the manifest.json. Just an FYI for those looking for a work around in the meantime.

j-fresquez avatar Jan 16 '23 17:01 j-fresquez

I just forked, corrected and used my own. Will sync back once it's fixed on main.

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: tr4j3dy @.> Sent: Monday, January 16, 2023 12:10:14 PM To: mcnutter1/homeasssitant-schlage @.> Cc: Greg @.>; Comment @.> Subject: Re: [mcnutter1/homeasssitant-schlage] manifest file typo (Issue #14)

+1 for typo on line 10 of the manifest.json, this prevents HACS from importing it and the integration loading properly. However, if you want you can still install the integration manually by adding the schlage folder of the repo to your custom components folder and manually editing out the comma on line 10 of the manifest.json. Just an FYI for those looking for a work around in the meantime.

— Reply to this email directly, view it on GitHubhttps://github.com/mcnutter1/homeasssitant-schlage/issues/14#issuecomment-1384341502, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAAOORT4LL4P2WDOV6U7UWDWSV6HNANCNFSM6AAAAAATWFIXCM. You are receiving this because you commented.Message ID: @.***>

schergr avatar Jan 16 '23 21:01 schergr

Has this fix been placed into a new release? Do I need to remove\reinstall?

mikebol avatar Jan 28 '23 02:01 mikebol