search

guerrerotook / securitas-direct-new-api

This repository contains the new securitas direct API that can be integrated in Home Assistant
Apache License 2.0
79 stars 30 forks source link

configuration.yaml validation error #94

Closed luigixx73 closed 1 year ago

luigixx73 commented 1 year ago

In configuration.yaml

securitas: username: !secret securitas_direct_username password: !secret securitas_direct_password code: ---------> EXPECTED STRING VALUE country: ES

If you set a string value for code in configuration.yaml or secrets.yaml and go to validation, HA will show an error as by code init.py line # 66 is expecting a string.

It also affects to disarming function that doesn't work. Replacing line #66 with: vol.Optional(CONF_CODE, default=1234): int,

will define this parameter as integer so it will work for validation and disarming.

B.Regards!!

guerrerotook commented 1 year ago

Oh yes, this is because the internal state of HA had the configuration as integer, but I have to change it to int. You may need to remove the configuration, the integration and reinstall again. Sorry for the confusion but there was some users that has codes that starts with zeros.

luigixx73 commented 1 year ago

Good job Luis, thanks

De: Luis Guerrero @.> Enviado: miércoles, 23 de noviembre de 2022 17:29 Para: guerrerotook/securitas-direct-new-api @.> Cc: luigixx73 @.>; Author @.> Asunto: Re: [guerrerotook/securitas-direct-new-api] configuration.yaml validation error (Issue #94)

Oh yes, this is because the internal state of HA had the configuration as integer, but I have to change it to int. You may need to remove the configuration, the integration and reinstall again. Sorry for the confusion but there was some users that has codes that starts with zeros.

— Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fguerrerotook%2Fsecuritas-direct-new-api%2Fissues%2F94%23issuecomment-1325344291&data=05%7C01%7C%7Cc6aa6c78d7ff40c4816008dacd6fe025%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638048177621784951%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=IKJusWvpK8DvhCXc4xRM6h7VMboCURWH3mf3fN6r8Bc%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAYWZDXQQAFWSS5FZWG46IDDWJZA6BANCNFSM6AAAAAARYAQIZM&data=05%7C01%7C%7Cc6aa6c78d7ff40c4816008dacd6fe025%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638048177621784951%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=kLsYpXSb80bgih1%2BDKYGKJqnSe1sk01p1f12i6X8AV0%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>

PozoSer commented 1 year ago

I'm having the same issue. Just installed it and it expects a string.