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

Feat report#14

Open
Westixy wants to merge 41 commits into
ES-Community:masterfrom
Westixy-FORK:feat-report
Open

Feat report#14
Westixy wants to merge 41 commits into
ES-Community:masterfrom
Westixy-FORK:feat-report

Conversation

@Westixy

@Westixy Westixy commented Aug 5, 2018

Copy link
Copy Markdown
Member

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

@Xstoudi Xstoudi left a comment

Copy link
Copy Markdown
Member

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 :)

Comment thread 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.`)

Copy link
Copy Markdown
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"

Comment thread 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(' ')}`)

Copy link
Copy Markdown
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
Copy Markdown
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

Comment thread 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.`)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

*débloquer

Comment thread 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\`.`)

Copy link
Copy Markdown
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

Comment thread 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`)

Copy link
Copy Markdown
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

Westixy commented Aug 15, 2018

Copy link
Copy Markdown
Member Author

Ajout de l'embed
image

@Xstoudi

Xstoudi commented Aug 17, 2018

Copy link
Copy Markdown
Member

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

Westixy commented Aug 17, 2018

Copy link
Copy Markdown
Member Author

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

@Xstoudi

Xstoudi commented Aug 20, 2018

Copy link
Copy Markdown
Member

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