petrleocompel / hikaxpro_hacs

HACS repository of Hikvision Ax Pro integration for home assistant
MIT License
40 stars 4 forks source link

Error to arm | HA 2024.6 compatibility #118

Closed infvito closed 2 months ago

infvito commented 3 months ago

Arming requires a code but none was given for alarm_control_panel.ax_pro

petrleocompel commented 3 months ago

Can you provide more info ? logs from HomeAssistant ? Version of HA you are using ? etc ?

infvito commented 3 months ago

Core 2024.6.2 Supervisor 2024.06.0 Operating System 12.3 Frontend 20240610.0 Ax pro v 1.2.9 build 240516

Registratore: homeassistant.components.websocket_api.http.connection

Fonte: components/websocket_api/commands.py:283

Integrazione: Home Assistant WebSocket API (documentazione, problemi)

Prima occorrenza: 10:45:07 (2 occorrenze)

Ultima registrazione: 23:39:49

[140680330512704] Arming requires a code but none was given for alarm_control_panel.ax_pro

[140679944869712] Arming requires a code but none was given for alarm_control_panel.ax_pro

the problem is in arming both at home and outside but disarming works after it has been entered from the hikvision app. I use it with an administrator user and or removed as mentioned also the installer

Il Mer 12 Giu 2024, 22:21 Petr Leo Compel @.***> ha scritto:

Can you provide more info ? logs from HomeAssistant ? Version of HA you are using ? etc ?

— Reply to this email directly, view it on GitHub https://github.com/petrleocompel/hikaxpro_hacs/issues/118#issuecomment-2163831730, or unsubscribe https://github.com/notifications/unsubscribe-auth/A63DLJ4HKCS6ABHXS3XGGRTZHCUULAVCNFSM6AAAAABJF7VP5CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRTHAZTCNZTGA . You are receiving this because you authored the thread.Message ID: @.***>

superbannato commented 3 months ago

I have the same problem.....

HA core 2024.6.2 Supervisor 2024.06.0 OS 12.3

Registratore: homeassistant.components.websocket_api.http.connection Fonte: components/websocket_api/commands.py:283 Integrazione: Home Assistant WebSocket API (documentazione, problemi) Prima occorrenza: 11:02:37 (2 occorrenze) Ultima registrazione: 12:34:09

[545893216560] Arming requires a code but none was given for alarm_control_panel.antifurto [548009426032] Arming requires a code but none was given for alarm_control_panel.antifurto

petrleocompel commented 3 months ago

What FW version do you have on AXPro?

Was your setup working before the update?

superbannato commented 3 months ago

My version of AxPro is V1.2.9 build 240516. Before the last Home Assistant update, it worked perfectly. In the configuration of the add-on (version 2.1.0-beta), I have always set the administrator user as recommended by the documentation. The only thing that works is the alarm disarm function.

infvito commented 3 months ago

I can confirm that the problem started after the update. I have now gone back the version and everything works again

superbannato commented 3 months ago

Will this issue be resolved in the next update of the add-on?

petrleocompel commented 3 months ago

I have to simulate this problem to fix it. Also since you commented on issue as I will comment, close or etc you will get notification + I am making releases and mentioning it in issues. Sadly due time constraints I will not tell you exact time "when".

I was looking at updates for HA and saw some redesign for control panels. But did not have time to upgrade yet. But 2.1.3 could solve this + one issue with user privilege.

superbannato commented 2 months ago

Thank you for your commitment and the work you're doing to fix this issue. I really appreciate your updates and notifications. Good luck with the simulations and the new releases!

HOMEGLN commented 2 months ago

More information: https://developers.home-assistant.io/blog/2024/05/22/alarm_control_panel_validation/

HOMEGLN commented 2 months ago

Standard Code

There is a temporary workaround (or permanent solution?) by entering a fictitious PIN code in the settings field of the alarm panel card. I used a PIN code that does not appear in my alarm and the integration works again. Previously, this field was empty for me.

petrleocompel commented 2 months ago

Temporary fix is done. Currently it should function as before but improvements regarding 2024 versions are not done.

petrleocompel commented 2 months ago

Version released 2.1.3

petrleocompel commented 2 months ago

Should be resolved. If anything wrong let me know and reopen the issue.