-
Notifications
You must be signed in to change notification settings - Fork 2
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
Adding a new Endpoint to a device does not work #43
Comments
What device ? Normally Endpoint information comes from the imported xml generated by OH. I have never added an Endpoint manually and it is discouraged. |
I know, but updating using an XML isn't available at the moment. This is what Chris Jackson said about XML import: "Hi. This isn't available in the new database. Just update the device type/id directly - it's the only thing that can be changed." Editing the device does not appear to allow me to add a new Endpoint. It goes through the motions but the new Endpoint does not appear in the list. The device in question is a Goap ZMNHSD DIN Rail Dimmer. |
Somebody changed the database for that device on Feb 27 but never requested review, indicating they were finished with the editing. It was therefore not flagged for review & approval. I just looked at the diff and it does not chow any changes made. Just editing & not asking for review may cause the device to be excluded from database updates though. |
That was probably me who made the edit on the 27th. The trouble is, if I add an Endpoint it's neither showing in the editor or in the diff even though it says it has been successfully created, so it feels like something is broken with the editor. I opened a ticket in OpenSmartHouse in February but had no response so I'm not sure how to proceed with this. |
Generally you should not add Endpoints manually anyway. The Endpoints are defined by the xml from the device . Leaving an entry in modified status can block it from database exports until it is approved. If an Endpoint is missing it was likely added in a firmware update and requires a new database entry to not break users with older firmware. @cdjackson perhaps there should be multiple edit levels with the lower level not allowing dangerous edits as described here but permitted for "Reviewers / Approvers". |
The device in question is a Goap/Qubino device. I had to modify the DB for a couple of other Qubino devices in the past; these required additional Endpoints due to peculiarities in how these devices send temperature etc. reports. They send them on a different Endpoint when polled to when they are unsolicited. The changes made the the other devices were made after discussions with Chris and were made using the previous DB editing mechanism. This change is pretty much doing the same thing for (yet) another Qubino device that is showing similar behaviour. |
It could be a breaking change because users could see extra non-functional ( for them) channels. I have personally had that happen with a DB entry that expected a temperature sensor that my device was lacking. If the website functionality is broken, then Chris could take a look. I have not been deeply involved with that part of things other than some backend scripting for the database. |
If I edit a device and add a new Endpoint, a pop-up a indicates that this has been created successfully. However the new Endpoint does not show in the Endpoint list so I am unable to add any Command classes or Channels. I have tried the usual refreshing the browser etc. Browser is latest Firefox.
The text was updated successfully, but these errors were encountered: