manager ne présente rien

[Sotoamino]

Bonjour. découvrant votre Package, j'ai voulu appliquer votre exemple pour les préfixs, mais rien n'apparait sur la page de management du bot, peu importe le serveur.

client.prefixes = JSON.parse(fs.readFileSync('./data/config/prefixes.json'))

const validatePrefix = prefix => prefix.length <= 5;guild, value) => discordClient.prefixes[guild.id] = value; 

const getPrefix = (discordClient, guild) => discordClient.prefixes[guild.id] || '!'; 

client.dashboard.addTextInput('Prefix', 'The prefix that is added to discord messages in order to invoke commands.', validatePrefix, setPrefix, getPrefix);

je ne sais pas si ce soucis viens de votre package ou autre. Auriez vous une solution ?

[Sotoamino]

Après recherche, je penses avoir trouvé le soucis. La page de manage semble manquant (dossier views). Possible de le rajouter ? ^^

[SimonLeclere]

La page de manage est dans le fichier guild.js normalement

[Sotoamino]

et bien j'ai l'impression qu'elle n'est pas chargé. Comme je vous ai dit, quand je veux éditer un serveur, j'ai une page blanche qui apparait :/

[SimonLeclere]

Avez vous spécifié le paramètre client secret ?

[SimonLeclere]

J'arrive à reproduire le problème, je vais chercher d'où ca vient

[SimonLeclere]

Apparemment il manque une parenthèse dans le dossier guild.js, je regarde ca quand j'ai le temps

[cestef]

L'erreur vient de ma dernière PR, désolé de ne pas avoir vu cette erreur avant. Elle vient du fichier routes/manage.js à la ligne 12. Je cherche encore pourquoi mais EJS ne render rien pour ce fichier 🤔

[cestef]

Et c'est à ce moment que je m'en veux de pas avoir ajouté d'error handling 😭 Je rajoute ça dans une prochaine PR

[SimonLeclere]

J'ai ajouté un log pour l'erreur lors du rendu de fichiers, je cherche la parenthèse mais c'est long haha

[SimonLeclere]

Elle vient du fichier routes/manage.js à la ligne 12. Moi j'ai cru comprendre que ca venait plutôt du fichier guild.ejs

Je sais pas trop pourquoi à la ligne 25 une partie du templating est pas interprété

[cestef]

Je viens de remarquer aussi haha, j'ai essayé EJS-lint comme ils conseillent mais ça me met une erreur bizarre:

Unexpected token (1:12) in views/guild.ejs
<%- await include("partials/header", { bot, user, title: `Settings | ${guild.name}`, is_logged, hasClientSecret: true }) %>

[SimonLeclere]

Ouais c'est pas toujours très clair

[cestef]

l'erreur vient de la boucle settings.forEach

[cestef]

Il interprète une accolade comme la fermeture de la boucle ducoup il pense qu'il manque une parenthèse après

[SimonLeclere]

Je vais regarder

[SimonLeclere]

Tu sais quelle accolade est mal interprétée ?

[cestef]

J'essaie de trouver x)

[cestef]

ça vient de ce qu'il y a à l'intérieur des ifs j'ai l'impression

[SimonLeclere]

Je vais tester chaque if un par un

[cestef]

Ok j'ai trouvé, ça vient du await setting.get

[SimonLeclere]

J'ai l'impression aussi

[SimonLeclere]

Je vais voir ce que ca renvoie

[cestef]

Ces lignes là ducoup: 18, 26, 34-37

[cestef]

si on retire le await à chaque fois tout revient à la normale

[SimonLeclere]

Le problèle vient des await oui, mais ce serait pas mal que les getters et setters restent asynchrones

[cestef]

J'ai une erreur différente:

await is only valid in async functions and the top level bodies of modules in /Users/cstef/Documents/Projects/discord-easy-dashboard/views/guild.ejs while compiling ejs If the above error is not helpful, you may want to try EJS-Lint: https://github.com/RyanZim/EJS-Lint

[cestef]

Il faut wrap le body dans un (async () => {})() je pense

[SimonLeclere]

Jsp si ca fonctionne en ejs, c'est plutôt pour un single-file-app ce genre de méthode

[SimonLeclere]

xD

[cestef]

[cestef]

si tu ajoutes async dans le forEach

[cestef]

enfin réparé !

[cestef]

à moitié je crois enfait x) je re regarde après manger, j'ai remplacé le .forEach par un for (const setting of settings) {}

[SimonLeclere]

Okay merci beaucoup, je t'avoue que je suis en voiture là c'est pas très pratique x) Encore merci pour ton aide

[SimonLeclere]

@cstefFlexin J'espère que t'es sûr de toi parceque j'ai pas trop la possibilité de vérifier si tout fonctionne haha ;) Je merge et je publie une nouvelle version.

[SimonLeclere]

@Sotoamino Essaye de mettre à jour le module et tout devrait fonctionner !

[Sotoamino]

@Sotoamino Essaye de mettre à jour le module et tout devrait fonctionner !

J'ai essayé et tout marche niquel. Merci à vous ^^