nielsfaber / alarmo-card

Home Assistant card for controlling the Alarmo component
98 stars 18 forks source link

Keypad not shown in HA Companion App #14

Closed ghost closed 2 years ago

ghost commented 2 years ago

Checklist

Expected behavior

Keypad is shown when armed and needed when option 'Keep the keypad always visible, also when no code input is required.' is inactive.

Actual behavior

Since version 1.2.0 the keypad in the HA Companion App (Android) is not shown, when Alarmo is armed and a pincode is needed. It only appears, when option 'Keep the keypad always visible, also when no code input is required.' is activated. It works like expected when using a webbrowser like Chrome and Firefox.

Steps to Reproduce

Screenshot_20211128-180335_Home Assistant

StevenBoelhouwer commented 2 years ago

I have the "Keep the keypad always visible, also when no code input is required." active and I'm not seeing the keypad "always". It only shows when an alarm is armed. That would be your intended way of working so it look likes its working is a bit mixed up.

pepe59 commented 2 years ago

Same here.

nielsfaber commented 2 years ago

I can confirm this issue:

I have the "Keep the keypad always visible, also when no code input is required." active and I'm not seeing the keypad "always".

I overlooked this case when I was testing it. I have to make some more changes to cover this function.

Otherwise the issue is not really clear to me. For me all works as I would expect it. Is the issue specific to the companion app? If so, it’s likely a matter of caching, so you are still seeing the old card.

pepe59 commented 2 years ago

Here I have this problem on all devices: Chrome browser on PC, Android tablet and phone with companion application. I've tried clearing the cache and that doesn't help.

StevenBoelhouwer commented 2 years ago

I can confirm this issue:

I have the "Keep the keypad always visible, also when no code input is required." active and I'm not seeing the keypad "always".

I overlooked this case when I was testing it. I have to make some more changes to cover this function.

Otherwise the issue is not really clear to me. For me all works as I would expect it. Is the issue specific to the companion app? If so, it’s likely a matter of caching, so you are still seeing the old card.

Let me know if you need help testing!

nielsfaber commented 2 years ago

Please update the card to v1.2.1. This version should have the problem fixed.