ioBroker / ioBroker.ping

Pings configured IPs for ioBroker
MIT License
21 stars 18 forks source link

Missing rooms from CCU in selection list #22

Closed JayVee2 closed 4 years ago

JayVee2 commented 4 years ago

The rooms selection list doesn't list all rooms defined in CCU. Especially changes seem not to be updated. CCU3 and iobroker already rebooted. Adapter restarted. Any idea? -jayvee

Apollon77 commented 4 years ago

What this has to do with the ping adapter? Created issue in wrong repository?

JayVee2 commented 4 years ago

The ping adapter allows in column 3 of the ip address list to assign a room to each ip address.

I'm using the adapter to verify my whole IT infrastructure and create a live HTML report from this to be shown in vis/mediola. If a significant device is not responsive I'm sending an TTS message to sonos/alexa. ping adapter works fine for me to detect unstable WLAN devices or unintential LAN unplugging in combination with an add-on script.

BUT: In the meantime I have 36 IP addresses to be checked (heating gateways, broadlink IRs, gosunds, alexas, sonos' ...). It is useful to assign a room to each IP address in the list for clarity.

AND: As written in another thread, it would be great if the list of IP addresses would be sorted by either name or IP address (anthing fine to me). Currently the order is given by order of input.

If I knew where the list is stored, a manual edit of the list would be good enough to solve my needs as the list doesn't change that often.

Apollon77 commented 4 years ago

Ok so Tobias about that the room list imported from cpu is not up to date in iobroker? Then still wrong repo because hm-rega is responsible to sync names and rooms with ccu when i remember correctly. Do still not ping ;-))

Alternatively just add rooms by yourself by using enums page in iobroker.

JayVee2 commented 4 years ago

Thanks for the hint with the enum page. I understand it's rega adapter problem, not a ping adapter problem.

However as this is a ping thread, is it very difficult to add an automatic sort to the ip address table on save? No user interface required at all. The next time it shows up it's shown sorted. That would solve it.

Apollon77 commented 4 years ago

There is an extra issue for this correct ? ;-)

JayVee2 commented 4 years ago

Yes, #14

The issue has been opened more than a year ago. I'm unsure if there is still anybody to take care of.

Apollon77 commented 4 years ago

As soon as someone finds time it can be taken care of ... I would more sort by IP instead of name ... but effectively ... time is an issue.

We also take PRs ;-)

JayVee2 commented 4 years ago

PRs?

Apollon77 commented 4 years ago

Pull request. ... if you can write JavaScript Code, Change it yourself and propose the change ;-)

JayVee2 commented 4 years ago

So I did with other JS projects. But I haven’t started with IoBroker adapter programming yet and I don’t properly understand the code how the table is saved after editing. I already thought about an extension of the ping adapter with my HTML table I posted in the other thread.

Apollon77 commented 4 years ago

Sorting could be done in admin/index.html and index_m.html in the save method. Most likely at the end before the callback is called.

Apollon77 commented 4 years ago

For this no iobroker knowledge is needed. Just Browser JavaScript ;-)

JayVee2 commented 4 years ago

Where is index_m.html?

Apollon77 commented 4 years ago

If not in admin dir then ignore.

JayVee2 commented 4 years ago

We also take PRs ;-)

So, I did. What's next?

Apollon77 commented 4 years ago

As soon as Bluefox or me find time we will review it, comment or accept the change and release a new version. :-)