Skip to content
This repository has been archived by the owner on Jun 12, 2020. It is now read-only.

Feat report #14

Open
wants to merge 41 commits into
base: master
Choose a base branch
from
Open

Feat report #14

wants to merge 41 commits into from

Conversation

Westixy
Copy link
Member

@Westixy Westixy commented Aug 5, 2018

Implémentation possible de la commande report et unlock suite a l'issue #10

  • ajout d'une section roles dans la configuration
  • le fondateur doit avoir activer l'authentification en 2 facteurs pour pouvoir utiliser la fonctionnalité de modification de role (https://discordapi.com/permissions.html)
  • configuration d'un role Signalé empêchant l'écriture sur le serveur

L'implémentation est bien entendue a discuter

Copy link
Member

@Xstoudi Xstoudi left a 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 :)

}
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.`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"référançant"

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(' ')}`)
Copy link
Member

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'.

Copy link
Member Author

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

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.`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*débloquer

}
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\`.`)
Copy link
Member

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

}
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`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*liste des membres maybe ?

@Westixy
Copy link
Member Author

Westixy commented Aug 15, 2018

Ajout de l'embed
image

@Xstoudi
Copy link
Member

Xstoudi commented Aug 17, 2018

Deux remarques encore :

  1. J'afficherais le pseudo de la personne qui a signalé.
  2. J'afficherais le channel dans lequel le signalement a été effectué.
  3. J'afficherais les 5 derniers messages de la personne signalée dans le channel où elle a été signalée.

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

@Westixy
Copy link
Member Author

Westixy commented Aug 17, 2018

image
la personne qui a signalé est l'auteur donc celui qui a l'icone

@Xstoudi
Copy link
Member

Xstoudi commented Aug 20, 2018

Pour moi c'est ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants