classroom-monitor-bot
classroom-monitor-bot copied to clipboard
[FEATURE] Remove warning message from the CM Bot when a user updates their message.
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
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!
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.
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?
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
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?
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.