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

Device name limited to 15 characters #26

Closed
Eupeodes opened this issue Jan 11, 2023 · 6 comments · Fixed by #28
Closed

Device name limited to 15 characters #26

Eupeodes opened this issue Jan 11, 2023 · 6 comments · Fixed by #28

Comments

@Eupeodes
Copy link

Eupeodes commented Jan 11, 2023

Thanks for this integration. Gives options for much better push notifications than the Elro app (and I like having everything in HA).

When setting up some of my devices showed up with generic names like FIRE_ALARM while others gave the names I set in the Elro app. After some searching I found that those devices for which the name exceeded 15 characters had this problem. I guess this is hard or impossible to fix, so perhaps it would be good to metion the 15 character name limit in the readme.md.

@jbouwh
Copy link
Owner

jbouwh commented Jan 11, 2023

The 15 characters is a limit from the Elro connector. When changed in HA the name is also updated on the connector.

@Eupeodes
Copy link
Author

I had, in the Elro connector, names exceeding the 15 characters. Those were not synced to HA (those devices were given generic names. So it is sort of a limit, but not always strictly enforced by Elro. I still can set names longer than 15 characters in the Elro connector, but then the name in HA switches to something generic.

@jbouwh
Copy link
Owner

jbouwh commented Jan 12, 2023

Okay, interesting. In my device list (Apparatenlijst) on the Elro Connects app when I try to change a name using manage (Beheren) I can up the name but not use more than 15 characters.

@jbouwh
Copy link
Owner

jbouwh commented Jan 12, 2023

it would be good to metion the 15 character name limit in the readme.md.

That is a good suggestion.

@jbouwh jbouwh linked a pull request Jan 12, 2023 that will close this issue
@jbouwh
Copy link
Owner

jbouwh commented Jan 12, 2023

@Eupeodes Feel free to reopen if needed

@Eupeodes
Copy link
Author

No need to, documentation is updated, so looks good now! Thanks again for the work on this great integration.

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

Successfully merging a pull request may close this issue.

2 participants