Description

Adding tasks to the lib

The problem

Having to implement a task loop by yourself

The solution

Implement it

The current solution, if any

Checklist

• [x] - You have checked that there is no issue already mentioning this feature, open or closed
• [x] - Full details of the requested feature have been provided/shown

Additional context

Description

:D

Checklist

• [x] - All code changes have been tested, if any
• [ ] - The documentation has been updated accordingly for any code changes
• [x] - This PR fixes #96 an already existing issue
• [x] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [ ] - This PR fixes a bug
• [ ] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

Summary

No token added into Client will raise no error

Reproduction Steps

I wrote some code and ran it with no token and raises no error

Minimal Reproducible Code

-> __main__.py
from .bot import Bot

if __name__ == "__main__":
    Bot().login()


-> bot.py
from EpikCord import Client, Intents

class Bot(Client):
    def __init__(self, token: str):
        intents = Intents().all
        super().__init__(token, intents)

    async def on_ready(self):
        print("Ready")

Expected Results

Raise an error having no token

Actual Results

No error

Intents

All - Intents().all

System Information

Name: EpikCord.py
Version: 0.4.11
Summary: A Modern API wrapper for Discord, Intended for replacement of the now discontinued Discord.py library
Home-page: https://github.com/EpikCord/EpikCord.py
Author: EpikCord
Author-email:
License: MIT
Location: c:\users\jerry\appdata\local\programs\python\python310\lib\site-packages
Requires:
Required-by:

Checklist

• [X] This bug was found on the latest version of EpikCord
• [X] You have checked that there is no issue already mentioning this bug, open or closed
• [X] Full details of the bug have been provided/shown, including full tracebacks in case of exceptions
• [X] All bot tokens have been hidden/omitted from any screenshots or code, if any

Additional Context

No response

Description

File separation

Checklist

• [ ] - All code changes have been tested, if any
• [ ] - The documentation has been updated accordingly for any code changes
• [ ] - This PR fixes an already existing issue
• [x] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [ ] - This PR fixes a bug
• [ ] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

Summary

Some events are affected by a single change, affecting different things.

The Problem

So I was making the channel_create event handler, and I thought about how there's VoiceChannels and TextBasedChannels, and how it'd be nice to have a precise event for those. Discord.py had this with their "private_channel_create", which is whenever a DMChannelis created, but then you have channel_create event which is very broad and could cause issues if some nefarious people spam created DMChannels and the code would error, spamming the console.

The Ideal Solution

To have separate, precise events for each respectful type of change that could've happened within the broad event.

The Current Solution

No response

Checklist

• [X] This bug was found on the latest version of EpikCord
• [X] Full details of the requested feature have been provided/shown

Additional Context

No response

We should switch to Asynchronous Websockets with AIOHTTP, but the thing is I don't get how we'll use client.login, maybe that'll run the async version? I'm still baffled on how to do this one.

Description

This PR adds latency functionality to the library

Formula = difference between heartbeat and heartbeat_ack, measured in ms for more precision

Checklist

• [ ] - All code changes have been tested, if any
• [ ] - The documentation has been updated accordingly for any code changes
• [ ] - This PR fixes an already existing issue
• [x] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [ ] - This PR fixes a bug
• [ ] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

Description

I have finished adding dict.get(KEY) @TheUntraceable

Checklist

• [ ] - All code changes have been tested, if any
• [ ] - The documentation has been updated accordingly for any code changes
• [x] - This PR fixes an already existing issue #43
• [ ] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [x] - This PR fixes a bug
• [ ] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

Summary

I used the incorrect method to get the value of a key from the data dict.

Reproduction Steps

Literally anything.

Minimal Reproducible Code

Anything.

Expected Results

No errors to be thrown

Actual Results

Errors are thrown

Intents

All.

System Information

It won't run on anything.

Checklist

• [X] This bug was found on the latest version of EpikCord
• [X] You have checked that there is no issue already mentioning this bug, open or closed
• [X] Full details of the bug have been provided/shown, including full tracebacks in case of exceptions
• [X] All bot tokens have been hidden/omitted from any screenshots or code, if any

Additional Context

I need to replace all mentions of dict[key] with dict.get(key) to suppress KeyErrors.

Description

Just add some few major and minor improvements like implement :class: StickerItem and improve setup.py

Checklist

• [ ] - All code changes have been tested, if any
• [ ] - The documentation has been updated accordingly for any code changes
• [ ] - This PR fixes an already existing issue
• [x] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [ ] - This PR fixes a bug
• [x] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

Description

^ Hard Work for @TheUntraceable because he will need to update his PR 🤣

Checklist

• [ ] - All code changes have been tested, if any
• [ ] - The documentation has been updated accordingly for any code changes
• [ ] - This PR fixes an already existing issue
• [x] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [ ] - This PR fixes a bug
• [ ] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

Description

The library is very messy and outdated. It needs a rewrite. It has been a few months since I have actively worked on the library, and so I don't know where exactly I am now. Restarting the library will solve all these issues.

Checklist

• [ ] - All code changes have been tested, if any
• [ ] - The documentation has been updated accordingly for any code changes
• [ ] - This PR fixes an already existing issue
• [x] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [ ] - This PR fixes a bug
• [x] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Description

This PR contributes the following

1. Adds _channel_update and _channel_delete

Checklist

• [ ] - All code changes have been tested, if any
• [x] - The documentation has been updated accordingly for any code changes
• [ ] - This PR fixes an already existing issue
• [x] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
• [ ] - This PR fixes a bug
• [ ] - This PR is a breaking change (removes/renames functions, parameters etc.)
• [ ] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

The Presence class is meant to be used by users, but it can also be received, and so I need to either make the user use the respective arguments to set the Activity and Status, and then use the Presence class for receiving or make a new class, which seems rather redundant.

Summary

The callback for a command within a section has never received the self parameter.

Reproduction Steps

Make a command within a section. Call that command on Discord

Minimal Reproducible Code

No response

Expected Results

For the command callback to succeed

Actual Results

It fails.

Intents

No response

System Information

Latest

Checklist

• [X] This bug was found on the latest version of EpikCord
• [X] You have checked that there is no issue already mentioning this bug, open or closed
• [X] Full details of the bug have been provided/shown, including full tracebacks in case of exceptions
• [X] All bot tokens have been hidden/omitted from any screenshots or code, if any

Additional Context

No response

We should change Id's from strings to integers. This is because it is more of a hassle for developers to do their own stuff with the snowflake. They would have to do (int(snowflake) & 0x3E0000) >> 17, if they wanted to get the Internal Worker Id, rather ugly.

Summary

The goal of this feature is to make part of the code-base more reliable using a test suite that validate the behavior or given object/functions

The Problem

There is currently no test for any feature of the whole library. This means that there is a high risk of undefined behavior, especially considering #193

The Ideal Solution

Partial coverage from unit-testing and integration testing of feature that are not bound to discord API.

The Current Solution

No response

Additional Context

No response