Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

[FEATURE] Remove warning message from the CM Bot when a user updates their message. #150

Open
unnikrishnan2002 opened this issue Jan 1, 2022 · 6 comments

Comments

@unnikrishnan2002
Copy link
Contributor

Is your feature request related to a problem? Please describe.

The Bot gives warning when inclusive language is not used. But the warning message is not removed after the user updates their message accordingly.

Describe the solution you'd like

The Bot should remove the warning message after the user updates their message

Describe alternatives you've considered

No response

Additional context

No response

@github-actions
Copy link

github-actions bot commented Jan 1, 2022

Hello 👋 Thank you very much for raising an issue 🙌 The maintainers will get back to you soon for discussion over the issue! Thank you for your contributions. 🚀

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server - we help and encourage each other to contribute to open source little and often 🤓 Any questions, let us know!

@genzyy
Copy link
Contributor

genzyy commented Feb 24, 2022

Hey, I have an idea to implement this. Let me know if I can work on it or is there a better way to do this,

const csgo_scam =
  "hello i leave from csgo and give all my inventory, the first 3 will get trade";

const csgo_res = csgo_scam.match(/csgo|trade|inventory/g);

console.log(csgo_res);

const ban_requirements = {
  csgo: 3, // [csgo, inventory, trade]
  discord1: 2, // [discord, nitro]
  discord2: 3, // [discord, nitro, steam]
  discord3: 3, // [discord, nitro, month]
  discord4: 3, // [referral, discord, nitro]
};

if (csgo_res.length === ban_requirements.csgo) {
  // message.delete();
  // or
  // ban(user);
  console.log("delete message");
}

// same for other scams ...
  • Line 2 matches given string with the given words using regex and returns all the words that matched.
  • So the returned array's length is same as number of words we matched, then its a scam message.

@Panquesito7
Copy link
Collaborator

Hey, I have an idea to implement this. Let me know if I can work on it or is there a better way to do this,

const csgo_scam =
  "hello i leave from csgo and give all my inventory, the first 3 will get trade";

const csgo_res = csgo_scam.match(/csgo|trade|inventory/g);

console.log(csgo_res);

const ban_requirements = {
  csgo: 3, // [csgo, inventory, trade]
  discord1: 2, // [discord, nitro]
  discord2: 3, // [discord, nitro, steam]
  discord3: 3, // [discord, nitro, month]
  discord4: 3, // [referral, discord, nitro]
};

if (csgo_res.length === ban_requirements.csgo) {
  // message.delete();
  // or
  // ban(user);
  console.log("delete message");
}

// same for other scams ...
  • Line 2 matches given string with the given words using regex and returns all the words that matched.
  • So the returned array's length is same as number of words we matched, then its a scam message.

Looks good! 🚀 What do you think, @kaiwalyakoparkar, @SiddhantKhisty?

@siddhant-khisty
Copy link
Contributor

Hey, I have an idea to implement this. Let me know if I can work on it or is there a better way to do this,

const csgo_scam =
  "hello i leave from csgo and give all my inventory, the first 3 will get trade";

const csgo_res = csgo_scam.match(/csgo|trade|inventory/g);

console.log(csgo_res);

const ban_requirements = {
  csgo: 3, // [csgo, inventory, trade]
  discord1: 2, // [discord, nitro]
  discord2: 3, // [discord, nitro, steam]
  discord3: 3, // [discord, nitro, month]
  discord4: 3, // [referral, discord, nitro]
};

if (csgo_res.length === ban_requirements.csgo) {
  // message.delete();
  // or
  // ban(user);
  console.log("delete message");
}

// same for other scams ...
  • Line 2 matches given string with the given words using regex and returns all the words that matched.
  • So the returned array's length is same as number of words we matched, then its a scam message.

Looks good! 🚀 What do you think, @kaiwalyakoparkar, @SiddhantKhisty?

@Panquesito7 correct me if I'm wrong, but this looks like a solution for a different issue entirely🙂
And we have solved the issue of scam links using becca bot

@kaiwalyakoparkar
Copy link
Contributor

Yeah @SiddhantKhisty I feel the same. I don't see any code which checks for updated message or similar. I might be wrong. @genzyy could you please explain it maybe?

@genzyy
Copy link
Contributor

genzyy commented Feb 24, 2022

My bad!, I mistakenly added this code on a different issue than which I intended this solution for. Really sorry for that.

I aimed this solution for #138.

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

No branches or pull requests

5 participants