search

LengoLabs / qbot

Qbot is an advanced, easy to setup, free, and unbranded Discord-Roblox ranking bot. If at any time during setting it up you need assistance, you can join the support server.
https://discord.gg/J47m7t4
MIT License
77 stars 138 forks source link

add rowifi support #67

Closed brandonapt closed 3 years ago

brandonapt commented 3 years ago

discord user+tag: brandonn#1234 discord id: 785248634458996767

adding: verification checks for rowifi api aswell

brandonapt commented 3 years ago

no idea, ill ask in the support server!

IeuanGol commented 3 years ago

Thanks

brandonapt commented 3 years ago

there's no api rate limit. Proof: https://brandon.has-no-bra.in/duQ10M

yogurtsyum commented 3 years ago

This is a nice pull request, but I've been considering moving the function that checks if the user is verified over to my API. I will update this pull request with what I choose.

brandonapt commented 3 years ago

I'm bumping this PR, since i've added group-defender integration into qbot.

things to add: in the setup guide/.env make a new line called webhookurl which is where group-defender sends abuse reports. this cannot be empty.

I'm adding this since it's highly requested and would fit nicely in the huge v6 rewrite

yogurtsyum commented 3 years ago

I think we should code an anti abuse system from scratch instead of one that uses a npm package. It will make qbot more lightweight and will allow us to customize and integrate it more.

yogurtsyum commented 3 years ago

Also, in the future, I recommend you make separate pull requests for if you are adding different features. Because then we can choose to push them individually.

TheShadowGamer commented 3 years ago

This is a nice pull request, but I've been considering moving the function that checks if the user is verified over to my API. I will update this pull request with what I choose.

If you did this, then I think all qbot instances would be ratelimited the same. Ex:

Now they both have a ratelimit of two, even though they each only used the command once.

BillyH-0167 commented 3 years ago

This wouldnt work since their api returns a boolean (true /false) not a string the PR i just did should fix that.