Ankündigung

Einklappen
Keine Ankündigung bisher.

Frage zur Plugin Entwicklung

Einklappen
X
 
  • Filter
  • Zeit
  • Anzeigen
Alles löschen
neue Beiträge

  • psilo
    antwortet
    Ich habe direkt den code kommentiert. Die sachen sind inline im pr

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    Hey!
    Prima, so langsam verstehe ich den git flow. Danke für deine Hilfe. Wo finde ich denn deine "Kleinigkeiten" die du kommentiert hast?
    Ich habe im PR nur den einen comment gefunden bzgl. der fehlenden requirements.txt

    Einen Kommentar schreiben:


  • psilo
    antwortet
    immer von deinem fork aus mittels neuen pull requests. direkt in den dev wirst du auch nach dem merge nicht dürfen. übliches verfahren.

    ich fliege mal über den code, kann aber nur schauen und nicht testen, da keine hardware.

    PS: habe mal ne reihe an kleinigkeiten kommentiert.. ein riesending ist das meiste nicht...
    Zuletzt geändert von psilo; 14.10.2019, 15:52.

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    Glaube jetzt hats geklappt. https://github.com/smarthomeNG/plugins/pull/289

    Nächste noob - Frage: Sollte das Plugin demnächst ma lim Dev sein... Mache ich die weitere Entwicklung dann innerhalb des Forks in meinem Repo?

    Einen Kommentar schreiben:


  • psilo
    antwortet
    thengsty eigentlich kannst Du nicht direkt einen Commit in den DEV pushen. Du musst einen PR machen... Direkt haben nur wir die Rechte.

    Ein Commit fügt die Änderung erstmal nur Deiner lokalen Versionierung hinzu.

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    ne, kein PR. Kann meinen commit dort nicht finden. Ich verstehe den gesamten Workflow aber auch nicht wirklich und die Anleitung ist so mäßig.
    Ist aber auch egal. Wer das Plugin haben will... gerne hier zugreifen: https://github.com/thengsty/sh-neato
    Ich werde gelegentlich auch neue Features einbauen.

    Einen Kommentar schreiben:


  • psilo
    antwortet
    Wo ist der PR im Develop? (https://github.com/smarthomeNG/plugins/pulls)

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    So, ich hab das neue NEATO- Plugin jetzt mal in Develop committed anhand der Anleitung.
    Das ist ehrlich gesagt mein erstes Github Projekt, weshalb ich nicht sicher bin ob ich alles richtig gemacht habe...
    Kann sich das mal jemand anschauen und mir sagen was ich alles falsch gemacht habe? ;-)

    Danke, Thomas

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    Guten Morgen,
    da eine erste, frühe Version des neuen Plugins "Neato vaccum robot" nun fertig ist, würde ich diese gerne im Dev-branch veröffentlichen und dort weiterentwickeln.
    Hier das aktuelle Repo. https://github.com/thengsty/sh-neato
    Wer kann mir denn ein paar Tipps geben wie ich das Ganze in die SH-Plugins dev branch überführe?

    Thomas

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    So, Fehler gefunden. Wen's interessiert:

    Item Werte zuweisen über:
    Code:
     
     item.property.value = 'TollerWert'
    funktioniert zwar, triggert aber nichts in Richtung SmartVisu und "lastUpdate" wird auch nicht aktualisiert.

    Nutz man:
    Code:
    value = 'TollerWert'
    Code:
    item(value)
    klappt es einwandfrei.

    Grüße,
    Thomas

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    Glaube ich habe da so einen Verdacht: "last_Update" des Items wird nicht aktualisiert wenn das Update aus dem Plugin kommt. Möglicherweise wird deshalb nichts getriggert?

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    jo, korrekt. Das hatte ich inzwischen auch schon angepasst.
    Code:
     
     item.property.value = self.robot.isDocked
    Hat aber nicht geholfen. SmartVisu bekommt immernoch nichts von der Änderung mit.

    Einen Kommentar schreiben:


  • Msinn
    antwortet
    Was meinst Du mit _value?

    Mit _ beginnende Werte eines Objektes sind NICHT zu Zugriff von außerhalb des Objektes gedacht. Bitte verwende die offiziellen Methoden des Item Objektes um den Wert zu setzen.

    Einen Kommentar schreiben:


  • thengsty
    antwortet
    Ja, das hatte ich auch schon (erfolglos) versucht. :-(
    Danke!

    Einen Kommentar schreiben:


  • Tontechniker
    antwortet
    Hallo thengsty,
    hast Du schon mal mit
    "enforce_updates = true"
    versucht?
    Gruß
    Hans

    Einen Kommentar schreiben:

Lädt...
X