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

Fix all Blockly definitions of custom blocks #1598

Open
11 of 22 tasks
klein0r opened this issue Jun 7, 2024 · 9 comments
Open
11 of 22 tasks

Fix all Blockly definitions of custom blocks #1598

klein0r opened this issue Jun 7, 2024 · 9 comments

Comments

@klein0r
Copy link
Collaborator

klein0r commented Jun 7, 2024

@klein0r
Copy link
Collaborator Author

klein0r commented Jun 7, 2024

See https://github.com/google/blockly/releases/tag/blockly-v10.0.0 / google/blockly#7077

  • Blockly.WidgetDiv.DIV -> Blockly.WidgetDiv.getDiv()
  • Blockly.JavaScript.variableDB_ -> Blockly.JavaScript.nameDB_
  • use Blockly.JavaScript.quote_() to add quoted strings which escapes single quotes ' (optional)
  • Mutator-Blocks must be updated:
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));
}

@Lucky-ESA
Copy link

imap -> is fixed in next release >= 0.2.0
commit

@klein0r klein0r pinned this issue Jun 14, 2024
@klein0r
Copy link
Collaborator Author

klein0r commented Jun 14, 2024

@Lucky-ESA Thanks, just updated the generated code to fit the code style of all other blockly blocks: Lucky-ESA/ioBroker.imap#75

@BenAhrdt
Copy link

@klein0r was muss angepasst werden?
Adapter: LoRaWan

@Apollon77
Copy link
Contributor

@klein0r Wenn ein Adapter nicht angepasst werden kann wird da ab irgendwann was passieren das er nicht mhr geht?

@klein0r
Copy link
Collaborator Author

klein0r commented Jun 28, 2024

Eigentlich muss nichts angepasst werden. Nur Bausteine mit Mutator-Blöcken. Das sind aber super wenige Adapter.

@BenAhrdt
Copy link

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.
Also muss dazu nichts angepasst werden?
Ich habe im Adapter auch nur custom sendTo Blöcke drin.

@klein0r
Copy link
Collaborator Author

klein0r commented Jun 28, 2024

@BenAhrdt Ich habe einfach nur vor, alle Blockly-Bausteine mal anzuschauen und im gleichen Stil aufzubauen. Mehr nicht. 95% Kosmetik.

@crycode-de
Copy link
Contributor

@klein0r discord fixed in latest release v2.3.1

crycode-de/ioBroker.discord@2dc6b9a

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

5 participants