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

blockly: update to newest blockly, improve code and user doc #840

Merged
merged 22 commits into from
Nov 28, 2023

Conversation

onkelandy
Copy link
Member

No description provided.

@onkelandy onkelandy requested a review from psilo909 October 25, 2023 16:18
@psilo909
Copy link
Contributor

@onkelandy ich kanns am Samstag mal testen. Bzgl Code Review glaube ich aber nicht, dass das viel bringt. Das Update ist aber auf jdn Fall cool. Ich bringe unserem 10jährigen Sohn jetzt mal Mikrocontrollerprogrammierung mit einem blockorientierten Ansatz bei, da kann vielleicht auch bald Ideen für Logiken auf SHNG umsetzen ;-)

@onkelandy
Copy link
Member Author

Unbedingt gleich Sohnemann testen lassen ;)
Du bist als maintainer eingetragen, drum hab ich dich hier rein genommen hehe.
Es sind ne Menge Änderungen, der vorige Stand war wirklich ziemlich alt. Komisch ist, dass ich bei den return values aus den Item-Blocks die auskommentierte Variante nehmen musste, damit es funktioniert.

Eine Laden-Funktion hat es nie gegeben, oder? Wäre aber eigentlich auch noch machbar und sinnvoll oder was meint ihr?

@psilo909
Copy link
Contributor

So hab jetzt mal den PR gezogen und eingebaut. Schauma mal.. Vom Code habe ich auch keine Ahnung mehr - glaube aber auch nicht, dass da viel von mir gekommen war..

@msinn msinn changed the title Blockly Plugin: update to newest blockly, improve code and user doc blockly: update to newest blockly, improve code and user doc Oct 31, 2023
@psilo909
Copy link
Contributor

psilo909 commented Oct 31, 2023

mit bestehender config sieht das webif derzeit so aus?! mache ich was falsch?
image
image

@onkelandy
Copy link
Member Author

@psilo909 can we merge this?

@psilo909
Copy link
Contributor

@onkelandy Tuesday, ok?

@psilo909 psilo909 merged commit 494c819 into smarthomeNG:develop Nov 28, 2023
5 checks passed
@onkelandy onkelandy deleted the blockly branch December 29, 2023 19:11
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.

2 participants