Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Show default value for device config parameter #40

Closed
marcelveldt opened this issue Jul 4, 2024 · 4 comments
Closed

Show default value for device config parameter #40

marcelveldt opened this issue Jul 4, 2024 · 4 comments
Assignees
@wendevlin
Labels
frontend This bug or request (primary) involves the Home Assistant frontend zwave-certification Required for Z-Wave certification

Comments

@marcelveldt
Copy link
Collaborator

In the frontend panel where a user can configure the device configuration parameters, we need to show the default value.
This information should also be present in the current data (just like the min/max values) but its probably not yet exposed in the websocket command. It should be present in the value metadata.
@marcelveldt marcelveldt converted this from a draft issue Jul 4, 2024
@marcelveldt marcelveldt added the frontend This bug or request (primary) involves the Home Assistant frontend label Jul 4, 2024
@AlCalzone AlCalzone added the zwave-certification Required for Z-Wave certification label Sep 5, 2024
@wendevlin wendevlin self-assigned this Sep 5, 2024
@marcelveldt
Copy link
Collaborator Author

backend PR: home-assistant/core#125343

@wendevlin
Copy link
Collaborator

wendevlin commented Sep 9, 2024 •
edited
Loading

Frontend PR: home-assistant/frontend#21990

@AlCalzone
Copy link
Member

@wendevlin is that the right PR? Seems to be about resetting all parameters --> #39

@wendevlin wendevlin mentioned this issue Sep 12, 2024
Closed
9 tasks
@wendevlin
Copy link
Collaborator

@wendevlin is that the right PR? Seems to be about resetting all parameters --> #39

I updated the title of the PR now. It does solve #39 and #40

@wendevlin wendevlin mentioned this issue Sep 16, 2024
Merged
9 tasks
@github-project-automation github-project-automation bot moved this from In Progress to Done/verify in Z-Wave JS: Road to Certification Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wendevlin wendevlin

Labels
frontend This bug or request (primary) involves the Home Assistant frontend zwave-certification Required for Z-Wave certification
Projects
Z-Wave JS: Road to Certification
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@marcelveldt @wendevlin @AlCalzone