Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update signalduino_protocols.hash #512

Closed
wants to merge 8 commits into from
Closed

Update signalduino_protocols.hash #512

wants to merge 8 commits into from

Conversation

HomeAutoUser
Copy link
Contributor

  • update hash
  • Please check if the PR fulfills these requirements
  • Tests for the changes have been added / modified (needed for for bug fixes / features)
  • commandref has been added / updated (needed for bug fixes / features)
  • CHANGED has been updated (needed for bug fixes / features)

Die Änderung 959b63d wurde nicht im dev-r33 vorgenommen und um diesen aktuell zu halten, dieser PR.

Ralf9 and others added 3 commits February 13, 2019 07:35
@Ralf9
added option reconstructBit to protocols.hash, update ID 87 + 88 (RFD…
3c5c47d 
…-FHEM#510)

added doc option reconstructBit
added reconstructBit to to the ids 87, 88 and 91.1 and changed formatting
added clientmodule	=> 'SD_Rollo_Keeloq' and preamble => 'P87#' to ID 87
added clientmodule	=> 'SD_Rollo_Keeloq' and preamble => 'P88#' to ID 88

Added SD_Rollo_Keeloq to $clientsSIGNALduino and %matchListSIGNALduino
Travis automatic update controls file. Build (1147) [skip ci]
b161df8
@HomeAutoUser
Update signalduino_protocols.hash
025b43c 
- update hash
@sidey79
Copy link
Contributor

sidey79 commented Feb 17, 2019

Ich approve diesen pr derzeit nicht, da der Zweig dev-r33 nicht mehr die aktuellen Anpassungen aus dem master-3.3.3 hat.

Sobald der Master auf 3.3.3 ist, können wir 3.3.4 oder 3.4 development starten :)

@sidey79 sidey79 added this to the 3.4 milestone Feb 17, 2019
@sidey79 sidey79 changed the base branch from dev-r33 to dev-r34 February 17, 2019 22:32
@HomeAutoUser HomeAutoUser mentioned this pull request Feb 18, 2019
Merged
1 task
@sidey79
Copy link
Contributor

sidey79 commented Feb 18, 2019

@HomeAutoUser

ich glaube beim Changelog stimmt es nicht richtig. Sieht auch doppelt mit dem anderen PR aus

@HomeAutoUser
Copy link
Contributor Author

ich glaube beim Changelog stimmt es nicht richtig. Sieht auch doppelt mit dem anderen PR aus

Ja, da fehlt ein ganzer "Block". Den können wir ja mit ergänzen, das wir dort wieder einen fortlaufenden Stand haben.

@sidey79
Copy link
Contributor

sidey79 commented Feb 18, 2019

@HomeAutoUser

Irgendwas ist jetzt hier mit Zeilenumbrüchen vermutlich schief gelaufen

@HomeAutoUser
Copy link
Contributor Author

@sidey79
das ist ja richtig bescheiden. Spielt es für den weiteren Verlauf eine Rolle?

Ich vermute mal, das wird vielleicht die Software sein welche man benutzt.
Die Konflikte schaute ich mir soeben mit Atom an und habe diese darüber "geklickt" und auf "save" gedrückt.

@sidey79
Copy link
Contributor

sidey79 commented Feb 20, 2019

@HomeAutoUser

Das Diff der Änderung ist halt leider jetzt riesig. Kriegst Du das wieder zurück gedreht?

@HomeAutoUser
Copy link
Contributor Author

@sidey79 ich habe die Commits zurückgezogen und so sind es nur die Daten von Ralf geändert werden sollten + das update der Hash Datei. Die Änderungen umfassen hauptzächlich das Keeloq Modul und das der User es testen kann. Nach Sichtung der Commits bzw. Änderungen welche gemacht werden sollen, so können wir auch den cut wählen und den PR löschen.

Grund:

  1. Zusammentragung ist leicht wiederbeschaffbar
  2. die Änderungen von Ralf werden EH nochmal gedreht, weil der Modulname GEÄNDERT werden muss. (er konnte zu dem damaligen Zeitpunkt nicht wissen, das wir es nochmal umbenennen müssen) Wir haben Festgestellt, das außer dem Rollo auch eine Remote von etwas ganz anderem auch dort erkannt wird, aber die Hardware NIX mit Rollos zu tun hat. Somit wird der Modulname nur SD_Keeloq und ich überarbeite soeben das Ganze nach der "erschreckenden" Entdeckung.

HomeAutoUser added a commit to HomeAutoUser/RFFHEM that referenced this pull request Feb 21, 2019
@HomeAutoUser
update protocols.hash + revised 00_SIGNALduino.pm
d4fdff5 
- revised 00_SIGNALduino.pm, rename modul (new modul for keeloq devices, not only rollo´s)
- revised protocols doc

PR RFD-FHEM#512 not used more!
@HomeAutoUser HomeAutoUser mentioned this pull request Feb 21, 2019
Merged
3 tasks
sidey79
sidey79 reviewed Feb 21, 2019
View reviewed changes
CHANGED
@@ -1,5 +1,3 @@
18.02.2019
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ist das richtig, dass wir den Eintrag hier wieder löschen?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich weiß nicht wo der - 18.02.2019 zustande kommt.
Nach Sichtung der einzelnen commits kann dieser Confliktbehaftete PR gelöscht werden, weil ich die Korrektur, welche eh gemacht werden musste, geschaffen habe im anderen PR wo keine Konflikte entstehen sollten.

Die Einarbeitung von reconstructBit bei den Protokollen habe ich noch nicht gesetzt, da ich das erst bei Fertigstellung der "LastBit" Problematik gesamt hinzufügen würde.

@sidey79
Copy link
Contributor

sidey79 commented Feb 21, 2019

Dann mach den PR zu, wenn wir ihn nicht mehr benötigen :)

@HomeAutoUser HomeAutoUser deleted the dev-r33_hash_update branch February 21, 2019 21:27
@sidey79 sidey79 mentioned this pull request Feb 21, 2019
Closed
@HomeAutoUser HomeAutoUser mentioned this pull request Feb 22, 2019
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sidey79 sidey79 sidey79 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.4
Development

Successfully merging this pull request may close these issues.
3 participants
@HomeAutoUser @sidey79 @Ralf9