Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajout d'un check pour media.discordapp.com #182

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

kumakuma215
Copy link

En gros, dès que quelqu'un poste un message qui contient media.discordapp.net, ça va automatiquement supprimer ce message et le renvoyer en remplaçant media.discordapp.net par cdn.discordapp.com. Je sais pas du tout si ça marche mdrrr j'ai pas testé mais ça devrait marcher

Copy link
Member

@noftaly noftaly left a comment

Choose a reason for hiding this comment

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

Hello ! merci pour la PR :)

  1. Il faudrait que ca ne fonctionne uniquement dans le salon "memes"
  2. Je serai d'avis de renvoyer que les liens après, sans supprimer le message ; plutôt que de renvoyer tout le message en supprimant l'original. Il renverrait les liens dans une réponse au message originel, sans mention
    Ca résout le pb des nitros qui peuvent envoyer un message de 2k+ caractères, mais le bot qui est limité à ≤2k.

Comment faire ca :

  • Utiliser ce regex = /https?:\/\/media\.discordapp\.net\/attachments\/\d{17,18}\/\S+\.\S+/gimu qui match un lien media.discordapp.net complet
  • utiliser const occurrences = message.content.match(leRegex) qui renvoie un array de toutes les occurrences
  • utiliser const transformedLinks = occurrences.map(link => link.replace('media.discordapp.net', 'cdn.discordapp.com'))
  • Le bot envoie une réponse qui dit (en gros)
**username** : les liens media.discordapp.net c'est a chier, voici les liens cdn.discordapp.com :
transformedLinks.join('\n')

SI c'est trop et t'as la flemme de le faire, je peux terminer la PR stv, ca devrait être rapide !

@@ -92,6 +93,14 @@ export default class MessageCreateListener extends Listener {
return false;
}

private async _checkMediaDiscordapp(message: GuildMessage): Promise<boolean> {
if (!/media\.discordapp\.net+/.test(message.content))
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
if (!/media\.discordapp\.net+/.test(message.content))
if (!message.content.includes('media.discordapp.net'))

private async _checkMediaDiscordapp(message: GuildMessage): Promise<boolean> {
if (!/media\.discordapp\.net+/.test(message.content))
return false;
await message.channel.send(message.content.replace(/media\.discordapp\.net+/, 'cdn.discordapp.com'));
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
await message.channel.send(message.content.replace(/media\.discordapp\.net+/, 'cdn.discordapp.com'));
await message.channel.send(message.content.replace(/media\.discordapp\.net/, 'cdn.discordapp.com'));

Le + en regex veut dire "ce qu'il y a avant (ici un "t"), 1 ou plusieurs fois" 😄 Donc la ca matchait ---.net mais aussi ---.netttt :)

Copy link
Author

Choose a reason for hiding this comment

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

Stylé stylé, je le fais quand j'suis chez oim.

@noftaly
Copy link
Member

noftaly commented Mar 18, 2022

Hello @kumakuma215 !
t'as pu arriver chez toi sans soucis ?
Si oui, tu penses continuer cette PR ou tu veux qu'on prenne le relai pour les changements qu'il reste ? 😄
Si non, je te souhaite bonne route

@kumakuma215
Copy link
Author

Haha tkt j'étais juste allé chercher du lait ^^. Non sinon j'pense pas avoir le temps de faire la PR je vous laisse le relai mdr, (pas le temps + flemme + ratio + cope + seethe)

@Romitou Romitou marked this pull request as draft March 22, 2023 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants