search

aendue / ioBroker.reolink

Reolink adapter for ioBroker
MIT License
21 stars 10 forks source link

Benennung der Datenpunkte #16

Closed KoMa1978 closed 1 year ago

KoMa1978 commented 1 year ago

Hi,

ist auch nur Kosmetik, aber die erstellten Datenpunkte heißen z.B.

ID: emailnotification

Hier fände ich es gut, zur besseren Lesbarkeit z.B. "EmailNotification" zu nutzen. Das wird bei einigen Datenpunkten ja auch schon gemacht.

Viel wichtiger wäre mir aber der Name, der hier "E-Mail Benachrichtigung" heißt. Hier wäre es klasse, wenn man den Namen erstellen würde aus "reolink.0.device.name" und dem Namen (z.B. CAM Eingang.Email-Benachrichtigung).

Der Daikin Adapter (https://github.com/Apollon77/ioBroker.daikin/blob/master/README.md) macht das z.B. und das erleichtert die Nutzung in Scripten enorm, da ansonsten im Blockly nur 6x "E-Mail Benachrichtigung" auf 1 oder 0 geschaltet werden und man nur raten kann welche Kamera gemeint ist.

KoMa1978

aendue commented 1 year ago

Hallo, danke für dein Feedback...schau ich mir an!

aendue commented 1 year ago

Hallo, im State EmailNotification wird jetzt der Name _Emailnotify verwendet.

Damit der neue EmailNotification State angelegt werden kann, muss der alte manuell gelöscht werden. Das liegt daran, dass der alte state über die io-package.json erstellt wurde und mittels deleteState() nicht gelöscht werden kann. Der neue State wird dynamisch angelegt.

Bitte mal testen. Danke!

KoMa1978 commented 1 year ago

Mache ich gerne am Sonntag. Wie kann ich den Adapter mit gleicher Versionsnummer (0.1.0) aktualisieren? Geht das einfach über einen erneuten Install oder muss ich den Adapter, die Instanzen und Datenpunkte erst komplett entfernen?

KoMa1978 commented 1 year ago

Ignoriere das von eben ... Hab den Adapter entfernt und neu installiert ...

Sieht seeeehr gut aus. Jetzt kann man im Blockly auch schön sehen, was man gerade ändert ;-)

image

aendue commented 1 year ago

Sehr gut...freut mich sehr! Issue mach ich dann zu. Danke