codex-team/editor.js

this.Editor.BlockManager.clear() should be awaited in renderFromHTML()

Open

#2 641 ouverte le 28 févr. 2024

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)TypeScript (26 602 stars) (1 985 forks)batch import
good first issue

Description

I believe this.Editor.BlockManager.clear() should be awaited in renderFromHTML() to prevent errors.

The following example highlights the issue: data.blocks will often be empty as a result of not awaiting clear().

await $editorApi.blocks.renderFromHTML(html);
const data = await $editorApi.saver.save();

Guide contributeur

Stack technique
typescript
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
async/await in TypeScriptbasic knowledge of Editor.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
The issue points to the file `src/components/modules/api/blocks.ts` at line 209, where `BlockManager.clear()` is called without `await`. The fix is to add `await` before that call. There are no linked PRs or assignees, so this is open for a newcomer. The issue also provides a clear reproduction example.