-
-
Notifications
You must be signed in to change notification settings - Fork 203
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feat] Allow to change provisioning entries name/loc even when after they have a node binded #3699
Comments
We are running into this. When we notice a typo or want to change the name or location of a SmartStart entry that is already assigned the edit button is not present. Implementing this would make it easy to update the name or location in Home Assistant by simply editing the SmartStart entry then excluding the device and SmartStart would then re-include the device using the updated name/location. Thank You |
@AlCalzone Should this be possible? |
I think it's fine. This info is just for the user anyways. |
@AlCalzone -- Just upgraded to v9.14.0 and tested this. Don't see the edit button active after upgrade on all our SmartStart entries which are already assigned to nodes. Have cleared our browser cache with same results. Should the edit button become active on in-use entries after this upgrade? Let me know if you need debug information and if so where to send it. Thanks! |
zwave-js-ui: 9.14.0 Re-confirmed browser (latest Chrome) cache is cleared. No change, no edit buttons on active entries. Running zwave-js-ui on Windows in HTTPS mode (self-signed) if that makes any difference. |
LOL sorry just saw that I pushed the commit after 9.14.0 release. Please wait for 9.14.1 or, if using docker, use |
Will wait for 9.14.1. Thanks for the confirmation.
From: Daniel Lando ***@***.***>
Sent: Tuesday, June 18, 2024 2:10 AM
To: zwave-js/zwave-js-ui ***@***.***>
Cc: Joseph Worrall ***@***.***>; Comment ***@***.***>
Subject: Re: [zwave-js/zwave-js-ui] [feat] Allow to change provisioning entries name/loc even when after they have a node binded (Issue #3699)
LOL sorry just saw that I pushed the commit after 9.14.0 release. Please wait for 9.14.1 or, if using docker, use master tag isntead
—
Reply to this email directly, view it on GitHub<#3699 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ALPZV3SEL6NZTCENZYLLNLLZH72NDAVCNFSM6AAAAABHI6LSK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZVGU4TMMRRGM>.
You are receiving this because you commented.Message ID: ***@***.******@***.***>>
|
Ref: #3694 (reply in thread)
The text was updated successfully, but these errors were encountered: