search

ciklik / visual-editor

Edit your page block by block
87 stars 26 forks source link

Master #5

Open capetrel opened 2 years ago

capetrel commented 2 years ago

Le commit 235d963 corrige l'erreur erreur en console Uncaught (in promise) TypeError ... visual-editor-error

En corrigeant le 'TypeError', le 'Warning' que l'on voit dans la capture disparait, donc je ne sais pas sur de mon fix.

Pour l'autre correction parle d'elle même et fait suite à l'issue que j'ai ouverte.

Grafikart commented 2 years ago

Cette erreur vient d'une non installation de React sur ton projet. Si ton projet est un projet laravel ou vite essaie d'installer react et react-dom et cela devrait résoudre le problème.

Si je me trompe je veux bien que tu me donne les étapes pour produire le bug que tu met en capture pour que je puisse tester le fix.

Le commit mentionné masque les erreurs (catch silencieux) donc je ne peux pas l'accepter en l'état.

capetrel commented 2 years ago

Bonjour Grafikart, désolé pour le catch silencieux, je ne savais pas quoi faire : console.log, .error, .warn, ou un autre truc ? Donc j'ai juste fait un return. Sinon c'est un projet Laravel avec Laravel-mix et j'utilise yarn comme gestionnaire de paquet. j'ai installé react et react-dom et yarn me sort une liste d'une dizaine de warning incorrect peer dependency, example :

warning "@boxraiser/visual-editor > @radix-ui/react-dialog > @radix-ui/react-compose-refs@0.1.0" has incorrect peer dependency "react@^16.8 || ^17.0".

et si je force la version de react et react-dom en ^17.0 il n'y en a que 2 : 

warning " > @boxraiser/visual-editor@0.0.20" has incorrect peer dependency "react@^18.0.0".".
warning " > @boxraiser/visual-editor@0.0.20" has incorrect peer dependency "react-dom@^18.0.0".

Et dans les 2 cas l'erreur subsiste. Je vais continuer à chercher, j'ai plus l'impression que le problème vient de mon projet, peut être un problème avec l'outil de build, un truc du genre (quand je fait un build de production il n'y a que le 'typeError' qui reste)

Grafikart commented 2 years ago

J'ai testé sur un projet normalement yarn add react react-dom installe la version 18. Tu peux ignorer les warning (j'essaierais de bump la version de regarder le problème et bump ces dépendances.