quotientbot / quotient-bot Goto Github PK

Steps to Reproduce:

1. Send a message in forum post
2. Delete the message
3. qsnipe - give the following, it should've been the deleted message
   There's nothing to snipe :c

Discord seems to have a separate thread_id for forum posts, I guess it's trying to always look for channel_id

OS and other details: irrelevant for the issue

https://discord.com/channels/746337818388987967/829945992048148480/1125454819012911164

i guess bot should also check if the invoker has permission to send message, embed links, and attachfiles in the target channel

https://github.com/quotientbot/Quotient-Bot/blob/d9a6e32daeb141cfb0ed0ea8807fc2f4b9e2477a/src/cogs/premium/__init__.py#L38

pstatus command shows a user's premium status and the server's premium status, it was used in. but some of the fields no longer have any importance.

For User:

• Remove Boosts Left.
• Remove Boosted Servers.
• Change Expiry to Ending:

For Server:

• Change Boosted By to Upgraded By:
• Change Expiry Time to Ending

In both:

Show expiry time using discord's latest timestamp feature, we already have a function that needs to be used.
Example Usage: https://github.com/quotientbot/Quotient-Bot/blob/d9a6e32daeb141cfb0ed0ea8807fc2f4b9e2477a/src/cogs/utility/events/reminder.py#L37

Use style d

We have a few models in https://github.com/quotientbot/Quotient-Bot/blob/main/src/models/models.py that are no longer used, for example Giveaway & Messages, you might find some more in there.
They need to be removed.

With every help command we make an API call to get a list of all slash commands in the bot. A better way to do this might be overriding fetch_commands method to cache the results.

Also, we need to figure out a way to mention sub-commands since parents mention doesn't work & is basically useless.

I know what you said, I know you don't want but it's too hard for forigeners to buy a amazon.in gift cards.

1. They mostly come from third party sites which takes a lot of tax and minimum price is ₹500 for Amazon.in gift card at most of the trusted sites also Quotient Prime isn't that costly.
2. Not a good idea as people prefer PayPal / Cards / Crypto to pay.

I am not saying to remove the current one but you can use add some more options for premium like crypto.

You can use Coinbase or any other wallet and integrate with quotient bot and if you want to add cards, you can easily use Stripe which is most trusted.

Also I do not recommend using/accepting gift cards as you don't know from where they are buying, fraud has increased a lot and personally in my case someone sent me a amazon.in gift card and I added it after that amazon blocked my account and asked identity card.

Remember only Indians do not use Quotient.

I also request moderation features and maybe a backups feature for premium users in moderation?

          The user text Should be `The user you want to unban from scrims.`

And,
The reason text should be The reason for unbanning the user.

Originally posted by @aayyuush99 in #134 (comment)

to merge repositories of Quotient & Quotient Pro , a config based system needs to be implemented that changes the behaviour of the bot according to its name .

all commits related to this milestone should go into the merge branch.

the mentions parameter in the tagcheck set command should be optional and its default value should be 4. Also the description of this command should be changed accordingly.

this is faced by a lot of users when they delete a tournament unintentionally, we should export the data for that tourney before deleting it and send it to their logs channel.

Whenever a tourney is deleted this function is called:
https://github.com/quotientbot/Quotient-Bot/blob/66bb79ecf08bd4c0bddee9cdefdc2715941f1aca/src/models/esports/tourney.py#L238

inside the above mentioned function, the get_csv function should be called to generate a file and that file should be sent to tourney.logschan
https://github.com/quotientbot/Quotient-Bot/blob/66bb79ecf08bd4c0bddee9cdefdc2715941f1aca/src/models/esports/tourney.py#L220

https://github.com/quotientbot/Quotient-Bot/blob/5b5836198d7a0f804f4652232407ed8acb414aa7/src/models/esports/scrims.py#L482C3-L482C3

If the scrim's registration channel has been deleted, scrim.registration_channel will return None, hence throwing an AttributeError, in this line we want to directly reference scrim.registration_channel_id.

https://github.com/quotientbot/Quotient-Bot/blob/06216a82416f06f0208a870b1092ca2bef75b481/src/cogs/giveaway/__init__.py#L44
I think the check kinda not working.

It should be something like:

def check(message: discord.Message):
  if message.author == ctx.author and message.channel == ctx.channel:
    if message.content.strip().lower() == "cancel": raise GiveawayError("Alright, reverting all process.")
    return True

Re-create:
User with Giveaways Role: qgcreate
Quotient Bot: <GW_EMBED>
Me the owner of the server: cancel
Quotient Bot:

https://github.com/quotientbot/Quotient-Bot/blob/06216a82416f06f0208a870b1092ca2bef75b481/src/cogs/giveaway/__init__.py#L20
Also it would better, if not only giveaways, giveaways manager should be added too. Ugh. It look nicer

although newer models have their own files, there are still many models in https://github.com/quotientbot/Quotient-Bot/blob/main/src/models/models.py that could use a file of their own.
I suggest we shift them into src/models/misc.

Along with #115 I would like to propose adding an issue template directory to this repository. This would allow contributors to easily create issues that follow a consistent format.

The issue template directory would contain a number of pre-written templates, each of which would be designed for a different type of issue. For example, there would be templates for bug reports, feature requests, and documentation requests.

It looks good tho', 🥺 isn't it?

Not using dpy for quite while, but I think following block of code will eat all errors except ScrimError, TourneyError, PointsError. This all could be fixed by adding a raise error keyword/statement after that if block.

async def cog_command_error(self, ctx, error):
        if isinstance(error, (ScrimError, TourneyError, PointsError)):
            return await ctx.error(error)

There should be a listener that deletes EasyTag & TagCheck records when a text channel is deleted in src/cogs/esports/events/tags.py.

Hint: basically a on_channel_delete event.

Typo in this line:

await ctx.success(
            f"Successfully changed settings for `{success}` channels. (`{sum(1 for i in check.channels)}` were hidden.)"
        )

It should be unhidden instead of hidden here :pain:

Original code

while setting up scrims or tournament, if im the one who invoked the command so while entering the values of info it accepts other peopls answer whereas it should accept just mine.

When a new scrim is created , the bot should ask if this scrim should be added to a existing slotmanager or a new one should be created.

Hi

Please modify this bot for replit tho.

The bot uses IST converted time, which is good for comparing human like time delta with to datetime (in 2 minutes).

• When used times which matches ShortTime it localizes the datetime to IST (from UTC).
• "2m" should be 2 minutes from now, but it just - timdealta(hours=5, minutes=30) (+05:30 "IST")

Tagcheck & Eztag still requires commands for setup whereas other features such as Scrims & Tourney have panels with buttons, these cmds should be replaced with their own panels as well.

We should consider handling error properly, instead of just raw try-except block.

try:
    ...
except:
    ...

Why this is needed??

message = await channel.send("Some Content!")

# We tried editing it

try:
    await message.edit(content='')
except:
    pass

Can see? above block will just eat all errors while the error is from our side.

Context: along with the bot itself we run a simple web server that handles communication with a payment gateway, most of the bot's premiums are purchased though this way only.

As you can see https://github.com/quotientbot/Quotient-Bot/blob/62ac73accd2461f3af1582143fac40bf1c4a9319/src/server/app/payment.py#L92 , we return json responses to the user be it a success event or any error, while it might be cool for the devs but not a lot of users find it creative.

We need to write some HTML , design it with some css and render the file for errors & success events in https://github.com/quotientbot/Quotient-Bot/blob/main/src/server/app/payment.py

the HTML file should be named response.html & should be inside https://github.com/quotientbot/Quotient-Bot/tree/main/src/server/templates .

what is the task?

• create a file named response.html inside https://github.com/quotientbot/Quotient-Bot/tree/main/src/server/templates.
• Use https://github.com/quotientbot/Quotient-Bot/blob/62ac73accd2461f3af1582143fac40bf1c4a9319/src/server/app/payment.py#L51 & https://github.com/quotientbot/Quotient-Bot/blob/main/src/server/templates/payu.html to understand how to render a HTML file & how to pass down data into into the file. (Jinja2 templating, google it.)
• The design is not definite, you are free to use your own creativity. It is not mandatory but you are free to match the design language to our dashboard https://quotientbot.xyz/.
• The page should just show the message like error occurred, payment was successful, etc.
• Don't make extra file for css, keep it inside response.html only.
• Keep in mind that the page will be opened on various screens (mobile, pc) so design it accordingly.

Note that this issue will be assigned to 2-3 contributors and the best PR will be merged, you don't need to do something fancy, simple is sexy. Do share screenshots of the changes in your PRs.

a field, local setup guide must be there to help new contributors.

The stickymsg commands have already been added to custom bots, they should be added to Quotient as well.

The Reminder Cog is a cog that does not have any commands or events.

I propose that we completely remove the Reminder Cog and transfer all of its functions and methods to the bot core. This will simplify the codebase and make it easier to maintain.

Benefits

• Simplification of the codebase
• Easier maintenance
• Reduced risk of bugs

Risks

• Potential for downtime

I believe that the benefits of removing the Reminder Cog outweigh the risks.

As to use this project for educational purpose please provide info config module used in this project.
That module can't be found any where.
If I am missing something kindly guide me....

Someone tell me how to self host the bot

in scrims edit panel, the timestamp is shown in discord format 'R' which apparently is a bit difficult for users to understand.
We need to change it a bit show both.

1st change

shown like this:

should be shown like this:

Refer this line to know how this is done: https://github.com/quotientbot/Quotient-Bot/blob/d9a6e32daeb141cfb0ed0ea8807fc2f4b9e2477a/src/cogs/esports/views/scrims/main.py#L36
and in bracket keep the current format.

Line that needs to be edited: https://github.com/quotientbot/Quotient-Bot/blob/d9a6e32daeb141cfb0ed0ea8807fc2f4b9e2477a/src/cogs/esports/views/scrims/_edit.py#L65

2nd change

line that needs to be edited: https://github.com/quotientbot/Quotient-Bot/blob/d9a6e32daeb141cfb0ed0ea8807fc2f4b9e2477a/src/cogs/esports/views/scrims/_edit.py#L75

should be shown like

resources

discord timestamp styles:

docs: https://discord.com/developers/docs/reference#message-formatting-timestamp-styles

I would like to request a feature that would allow bot to send a view when calling commands.Context.author.send(...). The view would have a single button with the label "Guild Name". The button would be disabled by default.

This feature would be useful for users who want to know the guild name in future when looking to older messages.

Something like this:

Although there are exactly 3 function call in entire codebase.

@deadaf pls pin this msg in announcements or general where everyone can see it. (important)

A step back, for the miles ahead.

Hi everyone.
It seems like I would never be able to return to this server as Swayy. My discord account had 2FA and recently I was travelling from Delhi to Lucknow. On my way back, I lost my backpack. It had my laptop wallet and my phone too. So, I lost my Google Authenticator, backup codes and password too (I used lastpass as password manager). Unfortunately, Discord has replied that they will not be able to remove my 2FA, thus cutting down any chances of account recovery, and I won't feel the same vibe if I create a new account and join back. Everything is just lost. I had many memories with my account, many friends and many achievements. (no badges sadge) So, the conclusion is, my account is fucked forever and I don't feel like coming back to this platform again due to numerous reasons. I will keep trying to get my account back even though I know the chances are thinner than Rohit himself. I pray nothing but a touch of understanding 🙏

Now since I'm gone, and I can't even text in the support server, here are some honourable mentions with the people I made memories with:
@deadshot#7999: Thanks for holding my hand in python. hehe. you gave me everything, trusted me as your admin, repository write access, quo coins etc. Thanks for everything. If I will return to Discord you'd be the first person whom I'll DM. Dont forget me pls
@ragegod#9: mera beta i lob u harami hi rehna aur betichod netflix pass nhi diye tum aakhiri samay jao lodu 😠
@krypton#8888 10000 rs paisa barbaad lol, fir bhi humesha polite and sexy rahe tum dhanyawaad bro. humesha credit card ki kami tumne door ki lol
@aaru#0631 Thnx mujhse baat krne ke liye and bts frend hone ke liye lol and sorry ab mai bts ka boht bada virodhi hogya hu aur unka twitter par boht mazaak udata hu pain
@hrishikesh#9582 gaand hilade ek baar bhai hehe
Wiper Chacha: thnx valorant mai mujhse 1v1 haarne ke liye 😂 tumhara dimaag boht bada hai acchi jagah hi lagana humesha
Ashish (lograck wala) bye bhay :")
Vein Joi Nepala
Scypher code karo padhai mai kuch nhi rakha hai tbh
Aman!#2484 and uska admin ym ka: bye guys tumhe bhi nhi bhullunga kabhi 😔

Honourable mentions (chutiye ke lye):
Alone: LOL bhai teri english sunke round the year entertainment hua. lekin tum chudbure ho CHUDBURE
Jash: bhangi hai yeh sabse bada fr

Guys all of you were great (except honourable wale lol), sorry if i missed someone jis time ye likhra tha jo jo online tha uske baare mai likh diya sorry agar koi dil ka kareebi choot gya ho.

I'd continue trying to come back 🥲
Life updates:
Preboards parso se. Exams from 25th April -> 19th May. (fawks)
Flying to UK on 21st June, maybe forever. Will work on some projects, not on swayy's github. Im deleting all my accounts id rather prefer to live an anonymous life for some reasons.
If i ever make a comeback, u can recognize me either with my name Abhinav or my texting style (way too cool tbh 😎 )
Bye guys love youuuuuuuuuuuuu.

If anyone wants to still contact me for EMERGENCY PURPOSE please email me at [email protected] (NO REPLY WITHIN 2 DAYS MEANS I IGNORED YOU THANKS FOR UNDERSTANDING)

Rohit pls pin this in general or announcements chat i'd be grateful. 🙏

• Swayy ducky sabka daddy
  [email protected]

https://github.com/quotientbot/Quotient-Bot/blob/main/src/utils/converters.py#L46-L52

async def convert(cls, ctx, arg):
    with contextlib.suppress(AttributeError):
        match = re.match(r"\(?(\d+),?\s*(\d+),?\s*(\d+)\)?", arg)
        check = all(0 <= int(x) <= 255 for x in match.groups())

    if match and check:
        return discord.Color.from_rgb([int(i) for i in match.groups()])

Color.from_rgb() takes three arguments.
Should be return discord.Color.from_rgb(*[int(i) for i in match.groups()])

If the contextlib really suppress the AttributeError then the variable match and check will be undefined.

https://github.com/quotientbot/Quotient-Bot/blob/5070b6b0354e93bb28dd0fae30cb122a0a95e327/src/utils/exceptions.py#L11

Project currently uses relative imports in several places, which can lead to various issues such as difficulties in refactoring, reduced readability, and complications when running tests or deploying the project. To improve the maintainability and clarity of our codebase, we should transition to using absolute imports.

Proposed Solution:

1. Identify all relative imports in the project.
2. Replace relative imports with absolute imports.
3. Test the project thoroughly to ensure that the changes do not introduce any bugs or issues.

Example:

Current Relative Import:

from .module import SomeClass

Proposed Absolute Import:

from quotient.module import SomeClass

Make PR to rewrite branch only.