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

Sensors need Unique ID values #59

Open
StealthChesnut opened this issue Oct 21, 2022 · 4 comments
Open

Sensors need Unique ID values #59

StealthChesnut opened this issue Oct 21, 2022 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@StealthChesnut
Copy link
Owner

Looking at sensors created by the integration in the settings area of Home assistant shows a message saying they can't be managed from the UI.

Add Unique IDs to prevent this.

@StealthChesnut StealthChesnut added the enhancement New feature or request label Oct 21, 2022
@StealthChesnut StealthChesnut self-assigned this Oct 21, 2022
@canton7
Copy link
Contributor

canton7 commented Mar 23, 2023

Fixed by #108?

@TonyM1958
Copy link

working on adding unique_ids to my mapping spread sheet along with entity names. Will post this when it's done if that helps?

@TonyM1958
Copy link

TonyM1958 commented Apr 4, 2023

I've added a simple formula to work out unique_ids in the attached spreadsheet.

H1 modbus map.xlsx

Once these are fixed, we can copy and paste values to stop them changing if the names are edited.

May need to consider consistency here with @nathanmarlor? Using the H3 config as a template, this currently sets the unique_ids to foxess_inv1_ in lower case with spaces changed to underscore. However, as far as I can tell, Nathan's code generates a unique id with a prefix of sensor.?

The problem, I think, if there are different unique_ids between these sets of code, anyone moving between the 2 implementations will loose their history?

@StealthChesnut
Copy link
Owner Author

I'd like this and Nathan's to be easy to migrate between, and I'm doing some testing to see what doesn't currently line up. Nathan did put some of the unique IDs in, trying to match up what's in this integration, but there may be the odd one here or there that doesn't match for some reason.

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

No branches or pull requests

3 participants