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.

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

New MQTT structure via lists kicks up parse error #2818

Closed
SirGoodenough opened this issue Aug 25, 2023 · 4 comments · Fixed by #3217
Closed

New MQTT structure via lists kicks up parse error #2818

SirGoodenough opened this issue Aug 25, 2023 · 4 comments · Fixed by #3217
Labels

Comments

@SirGoodenough
Copy link

The new MQTT Best Practice format complains when lists are used instead of dict.
Reference: https://www.home-assistant.io/integrations/mqtt/#manual-configured-mqtt-items

The following renders in HA and reloads/works fine:

Screenshot_20230825_102241

@alior101
Copy link

Text also doesnt parse

@janchrillesen
Copy link

The new syntax was added to HA in july in home-assistant/core#94305
The examples in the documentation have also been updated to the new format, that doesn't validate in the current version - https://www.home-assistant.io/integrations/sensor.mqtt/

@frenck
Copy link
Collaborator

frenck commented Oct 4, 2024

CleanShot 2024-10-04 at 15 29 31@2x

Works in the latest releases

@frenck frenck closed this as not planned Won't fix, can't repro, duplicate, stale Oct 4, 2024
@SirGoodenough
Copy link
Author

SirGoodenough commented Oct 5, 2024

This is a shot of today, right now and mine has an error.
image

Is it because I have multiple items listed with a - ?
I don't agree that it works today.
As far as HA the configuration works great to show a list of dicts. (or is it a dict. of lists, I can't remember)
HA 2024.10.1

Studio Code Server
Current version: 5.17.0 (Changelog)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants