search

elueckel / Unifi-Toolbox

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

Rename Device Blocker to Endpoint Blocker #52

Closed elueckel closed 2 years ago

elueckel commented 2 years ago

Discussed in https://github.com/elueckel/Unifi-Toolbox/discussions/51

Originally posted by **elueckel** January 5, 2022 Ich denke ich will den Device Blocker noch in Endpoint Blocker umbenennen (also nur Name). Endpoints sind Geräte während Device Teile von Unifi sind. Dann es er auch gleich mit dem Endpoint Monitor. Verzeichnis usw. würde ich gleiche lassen, sonst gibt es wieder Schmerzen beim Upgrade? Was denkst du @Brovning
elueckel commented 2 years ago

ok ... done ... auch der identifier ist jetzt UEB

elueckel commented 2 years ago

bin gespannt wie sehr das bei einem Update knallt :-)

Brovning commented 2 years ago

Du hast scheinbar beim Checkin vergessen die Nummer 52 anzugeben. Habe ich im Kommentar nachgeholt, damit der Checkin verlinkt ist.

Das Update funktionierte bei mir ohne Fehler. Lediglich die public function Aufrufe (sofern verwendet) müssen von UDB in UEB umbenannt werden. Bspw.: UDB_block($InstanzID, $device); → UEB_block($InstanzID, $device);

Brovning commented 2 years ago

@elueckel Du hast vergessen den Ordner umzubenennen. Im Ordner "UniFi Device Blocker" befindet sich nun der "UniFi Endpoint Blocker".

elueckel commented 2 years ago

Moin, ich hatte es nicht vergessen, sondern bewusst so gelassen, da ich damit in der Vergangenheit schon Probeleme hatte. Kann es nochmal testen und wann es keine Probleme macht, ändern wir es in einer 1.3.1 Am 2022-01-06 08:36, Brovning @.> schrieb: > > > Reopened #52 (https://github.com/elueckel/Unifi-Toolbox/issues/52). > > > > — > Reply to this email directly, view it on GitHub (https://github.com/elueckel/Unifi-Toolbox/issues/52#event-5850672434), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AHI57OZD7P332CMWTAAK7BTUUVBAFANCNFSM5LJU7VWQ). > You are receiving this because you were mentioned.Message ID: @.> > >

Brovning commented 2 years ago

Darf auch eine 1.4 inkl. TestSuite sein und was wir sonst noch bis dahin finden. ;-)

elueckel commented 2 years ago

klar ... mit dem Umzug auf einen Konfigurator könnte ich mir sogar eine 2.0 vorstellen da es eine grundsätzliche Änderung ist Am 2022-01-06 08:40, Brovning @.> schrieb: > > > Darf auch eine 1.4 inkl. TestSuite sein und was wir sonst noch bis dahin finden. ;-) > > > > — > Reply to this email directly, view it on GitHub (https://github.com/elueckel/Unifi-Toolbox/issues/52#issuecomment-1006347577), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AHI57O5VFO3GC6XOXMYZ75DUUVBN7ANCNFSM5LJU7VWQ). > You are receiving this because you were mentioned.Message ID: @.> > >

Brovning commented 2 years ago

Ja. sehe ich auch so. Konfigurator wäre eine 2.0. Alles andere, was kleinere Verbesserungen sind und auch die TestSuite kommen in eine 1.4.

Brovning commented 2 years ago

Habe es gestern noch geschafft alle nötigen PHP Funktionen für die TestSuite zu stubben und es hat ohne Fehler die TestSuite ausgeführt. Jetzt müssen "nur" noch die ganzen Funktionen gefüllt werden, so dass der UnifiController simuliert wird...

Brovning commented 2 years ago

Verzeichnisumbenennung siehe #57