ParadoxAlarmInterface/pai

Code for disarm not working

Open

#433 opened on Jan 30, 2024

View on GitHub
 (9 comments) (0 reactions) (0 assignees)Python (113 forks)auto 404
bughelp wantedhome-assistantto be confirmed

Repository metrics

Stars
 (414 stars)
PR merge metrics
 (PR metrics pending)

Description

The wiki describes it is possible to require a code for disarming through HA: https://github.com/ParadoxAlarmInterface/pai/wiki/Home-Assistant-integration#make-home-assistant-to-ask-for-a-code-to-armdisarm-dev-branch

First of all it mentions dev branch, but at the moment the dev branch is only 2 commits behind master (I assume that's stable), and those commits are unrelated to this. So the wiki should be updated.

In any case, adding the MQTT_HOMEASSISTANT_CODE in PAI config, and adding the customize code in my HA configuration.yaml doesn't change anything. Disarm still works and doesn't ask for a code.

PS: I also tried the development version of this HA addon, but with that one, arming/disarming doesn't work at all for me. The logs don't show any obvious errors.

Related: https://github.com/ParadoxAlarmInterface/pai/issues/194 https://github.com/ParadoxAlarmInterface/pai/issues/318

CC @yozik04

Contributor guide

Research direction
Inspect the configuration parsing for MQTT HOMEASSISTANT CODE and the Home Assistant discovery message generation to confirm the code parameter and code is optional flag are included, and compare with the wiki instructions.
Tech stack
python
Domain
backend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
GitPythonHome AssistantMQTT
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70