Skip to content

Conversation

@firestar300
Copy link
Contributor

for editor script and style

for editor script and style
@firestar300 firestar300 requested a review from Rahe November 18, 2021 14:29
@Rahe
Copy link
Member

Rahe commented Nov 18, 2021

Pourquoi nommer editor.css et editor.js ? on n'a que des slugs partout sur tous les autres entrées dans le switch.

@firestar300
Copy link
Contributor Author

firestar300 commented Nov 18, 2021

Pourquoi nommer editor.css et editor.js ? on n'a que des slugs partout sur tous les autres entrées dans le switch.

Les switch case ne correspondent pas aux noms générées dans le manifest et définies dans les entries de Webpack : https://github.com/BeAPI/beapi-frontend-framework/blob/master/config/entries.js

Je propose cette PR pour ne à devoir modifier dans chaque projet cette partie de code. On vient d'avoir le soucis sur CrossKnowledge, l'environnement de dev fonctionnait mais quand on a testé en environnement de production le style du back end était pété et on a perdu 30 minutes à savoir pourquoi.

De plus, cela ne correspond pas à ce qui est définie dans Editor.php :

@firestar300
Copy link
Contributor Author

@Rahe good pour toi du coup ?

@Rahe
Copy link
Member

Rahe commented Nov 24, 2021

Oui, on peut modifier comme ça

@firestar300 firestar300 merged commit 985456f into master Nov 24, 2021
@firestar300 firestar300 deleted the fix/editor-style-script-names branch November 24, 2021 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants