elueckel / Unifi-Toolbox

Modul for Symcon integrating UniFi Network devices
2 stars 2 forks source link

Korrektur für Einreichung #56

Closed elueckel closed 2 years ago

elueckel commented 2 years ago

die Einreichung Ihres Moduls UniFi Toolbox vom Konto elueckel wurde leider abgelehnt. Bitte überarbeiten Sie Ihre Einreichung noch einmal und reichen Sie diese erneut ein. Als Grund für die Ablehnung nannte das Symcon Team: In ApplyChanges erstellst du Variablen und setzt das benutzerdefinierte Profil von selbst erstellten Variablen. Dies liegt allerdings vollständig in der Hoheit des Benutzers und darf vom Modul nicht angepasst werden, siehe auch Punkt 11 der Reviewrichtlinien. Hierfür ist das Standardprofil vorgesehen. Daher solltest du deine Variable stattdessen per RegisterVariableBoolean (https://www.symcon.de/service/dokumentation/entwicklerbereich/sdk-tools/sdk-php/module/registervariableboolean/) erstellen. Damit kannst du alle Parameter von dir auch setzen, nur statt des benutzerdefinierten Profils das Standardprofil. Dies ist auch der vorgesehene Weg für Instanzen um die eigenen Variablen anzulegen.

elueckel commented 2 years ago

bin schon dran ...

Brovning commented 2 years ago

@elueckel bitte immer mit "#" und der Ticketnummer (in diesem Fall 56) die Checkins zu den Tickets verlinken.

elueckel commented 2 years ago

Geht klar … ich bin noch GitHub Neuling 👍

Am 06.01.2022 um 15:33 schrieb Brovning @.***>:

 @elueckel bitte immer mit "#" und der Ticketnummer (in diesem Fall 56) die Checkins zu den Tickets verlinken.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.