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

14_SD_WS09 und 14_Hideki in SVN übernehmen #200

Closed
2 tasks done
sidey79 opened this issue Dec 1, 2017 · 5 comments
Closed
2 tasks done

14_SD_WS09 und 14_Hideki in SVN übernehmen #200

sidey79 opened this issue Dec 1, 2017 · 5 comments

Comments

@sidey79
Copy link
Contributor

sidey79 commented Dec 1, 2017

Improvemnt from PR #199 in SVN übernehmen.

  • 14_Hideki reviewen
  • 14_SD_WS9 reviewen
@Ralf9
Copy link
Contributor

Ralf9 commented Dec 6, 2017

Wenn dieser commit im master ist, dann kann das 14_Hideki ins svn
05152f4

@Ralf9 Ralf9 added the Oregon label Dec 26, 2017
@Ralf9
Copy link
Contributor

Ralf9 commented Dec 26, 2017

Im Oregon Modul passt eine kleinigkeit nicht.
In der Zeile 1121 steht
Log3 $iohash, 4, "$name decoded Oregon: $val";
dies müsste aber heissen:
Log3 $name, 4, "$name decoded Oregon: $val";

Ist es ok, wenn ich diesen fix direkt ins master commite?

Ist dies ab Zeile 1111 so ok?

print "\nOREGON: Unknown: "; 
print "Type: ".$i->{type}.", ";
print "Value: ".$i->{current}."\n";

oder ist es so Log3 $iohash, 4, ".... besser?

@sidey79
Copy link
Contributor Author

sidey79 commented Dec 26, 2017

@Ralf9

Man kann den Hash oder den Namen übergeben. Beides ist richtig. Wird der Hash auf die Geräteinstanz übergeben, wird das in der Funktion umgewandelt in den Namen.

Bezüglich commit in den Master. Spricht etwas dagegen es in dev-r33 zu committen? Ich nehme an: nein.

@Ralf9
Copy link
Contributor

Ralf9 commented Dec 26, 2017

Habe es ins dev-r33 zu committet
0637031

@sidey79
Copy link
Contributor Author

sidey79 commented Mar 29, 2018

Ich denke ich habe den gewünschten code übernommen... ich hoffe es zumindest

@sidey79 sidey79 closed this as completed Mar 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants