-
Notifications
You must be signed in to change notification settings - Fork 120
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
Fix all Blockly definitions of custom blocks #1598
Comments
See https://github.com/google/blockly/releases/tag/blockly-v10.0.0 / google/blockly#7077
if (typeof Blockly.icons === 'object') {
this.setMutator(new Blockly.icons.MutatorIcon(['telegram_ask_mutator'], this));
} else {
// Blockly 9.x
this.setMutator(new Blockly.Mutator(['telegram_ask_mutator'], this));
} |
imap -> is fixed in next release >= 0.2.0 |
@Lucky-ESA Thanks, just updated the generated code to fit the code style of all other blockly blocks: Lucky-ESA/ioBroker.imap#75 |
@klein0r was muss angepasst werden? |
@klein0r Wenn ein Adapter nicht angepasst werden kann wird da ab irgendwann was passieren das er nicht mhr geht? |
Eigentlich muss nichts angepasst werden. Nur Bausteine mit Mutator-Blöcken. Das sind aber super wenige Adapter. |
@klein0r Ich dachte es müsste was erledigt werden, da der Adapter oben in deiner Liste steht. |
@BenAhrdt Ich habe einfach nur vor, alle Blockly-Bausteine mal anzuschauen und im gleichen Stil aufzubauen. Mehr nicht. 95% Kosmetik. |
@klein0r discord fixed in latest release v2.3.1 |
The text was updated successfully, but these errors were encountered: