-
Notifications
You must be signed in to change notification settings - Fork 2
Feat report #14
base: master
Are you sure you want to change the base?
Feat report #14
Conversation
Some change and improvements
Listen `ready` event once, Fix ES-Community#3
…ubscribe Feature channel subscribe
Xstoudi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques remarques sur la typo.
Petite suggestion aussi : pas la peine de faire une commande de dé-signalement pour les modos : on se débrouille avec le rôle je pense que ça suffit :)
src/commands/report.js
Outdated
| } | ||
| const userMatch = userString.match(/<@!?(\d{18})>/) | ||
| if (userMatch === null) { | ||
| message.author.send(`Vous devez spécifier un utilisateur en le référançant avec \`@username\` pour le signaler.`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"référançant"
src/commands/report.js
Outdated
| message.guild.members | ||
| .filter(m => m.roles.find('name', config.roles.moderator) !== null) | ||
| .array().forEach(m => | ||
| m.send(`[SIGNALEMENT] \`${message.author.tag}\` a signalé \`${reportedUser.user.tag}\` pour la raison suivante :\n${reason.join(' ')}`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu peux envoyer dans le channel mentors et tu peux nous faire un ptit rich embed des familles si t'es déter'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Peut-être à l'occas pour l'embed
src/commands/report.js
Outdated
| args: [userString, ...args] | ||
| }) => { | ||
| if (message.member.roles.find('name', config.roles.moderator) === null) { | ||
| message.author.send(`Vous devez être un mentor pour déblocker un utilisateur.`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
*débloquer
src/commands/report.js
Outdated
| } | ||
| const userMatch = userString.match(/<@!?(\d{18})>/) | ||
| if (userMatch === null) { | ||
| message.author.send(`Vous devez spécifier un utilisateur en le référançant avec \`@username\`.`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pareil que plus haut, *référençant
src/commands/report.js
Outdated
| } | ||
| const reportedUser = message.guild.members.find('id', userMatch[1]) | ||
| if (reportedUser === null) { | ||
| message.author.send(`L'utilisateur avec l'id \`${userMatch[1]}\` n'a pas été trouvé dans la liste de membre`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
*liste des membres maybe ?
|
Deux remarques encore :
Le tout bien évidemment pour savoir sur qui taper pour les potentiels signalement abusifs, pour savoir où chercher et pour savoir quoi chercher :) (Oui, ça fait trois en fait) Merci @Westixy |
|
Pour moi c'est ok |


Implémentation possible de la commande report et unlock suite a l'issue #10
rolesdans la configurationSignaléempêchant l'écriture sur le serveurL'implémentation est bien entendue a discuter