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.

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

revised preamble ID 91 - #491633536 #588

Merged
merged 1 commit into from
May 13, 2019

Conversation

HomeAutoUser
Copy link
Contributor

  • 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)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  • What is the current behavior? (You can also link to an open issue here)
  • preamble not work in modul

- SD_ProtocolData.pm: ID 91.1 revised preamble to P91
RFD-FHEM/SIGNALduino_TOOL#7 (comment)
Copy link
Contributor

@sidey79 sidey79 left a comment

Choose a reason for hiding this comment

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

Irgendwie haben wir schon wieder Probleme mit den tabs oder?

@HomeAutoUser
Copy link
Contributor Author

smile
das war das einzige Protokoll was allgemein aus der Reihe tanzte und so erlaubte ich mir dies zu korrigieren.
Den "Kaffee 5er bekommste :-D "

@sidey79
Copy link
Contributor

sidey79 commented May 13, 2019

Sieht mir jetzt eher nach einem wilderen Tanz aus, aber für mich ists wohl heute zu spät:

image

@HomeAutoUser
Copy link
Contributor Author

Aus diesem Grunde hatte ich TAB gedrückt

3

Bei manchen Protokollen wird es warscheinlich bei allen Systemen gleich richtig angezeigt aber bei sehr wenigen nicht obwohl die Tabbreite gesetzt wurde.

Copy link
Contributor

@sidey79 sidey79 left a comment

Choose a reason for hiding this comment

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

Ich glaub die Sache mit den tabs muss ich nicht mehr verstehen oder? Ich muss es wohl einfach hinnehmen dass es hier unterschiedliche Auffassungen der Weite gibt. Keine Ahnung was wir da machen könnten.

@HomeAutoUser HomeAutoUser merged commit 2abd8fa into RFD-FHEM:dev-r34 May 13, 2019
@HomeAutoUser HomeAutoUser deleted the dev-r34_preamble branch May 13, 2019 21:02
@Ralf9
Copy link
Contributor

Ralf9 commented May 13, 2019

Es gibt dafür eine einfache Lösung, einfach den Kommentar am Anfang der SD_ProtocolData.pm beachten, dann passt es.

Between the keys and values ​​no tabs not equal to a width of 8 or please use spaces

@HomeAutoUser
Copy link
Contributor Author

2 is not 8

Zwischenablage-13

Schwamm drüber, gibt größere Bugs zu beheben ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants