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

Add some models classes #6

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Add some models classes #6

wants to merge 26 commits into from

Conversation

Gugu7264
Copy link

@Gugu7264 Gugu7264 commented Jan 3, 2022

This PR adds some models, updates .gitignore so my editor's config isn't pushed, and linted everything (some commits from master weren't linted, so I did)

nextcord/core/gateway/protocols/gateway.py Outdated Show resolved Hide resolved
nextcord/exceptions.py Outdated Show resolved Hide resolved
nextcord/types/allowed_mentions.py Outdated Show resolved Hide resolved
@@ -0,0 +1,179 @@
"""
The MIT License (MIT)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I honestly dislike the embed api, it looks super messy imo

Copy link
Author

@Gugu7264 Gugu7264 Jan 3, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I honestly dislike the embed api, it looks super messy imo

Oh? Why that? Because of a class per Proxy?

Or maybe I should add some set_* functions to the Embed class?

nextcord/types/protocols/abc.py Outdated Show resolved Hide resolved
nextcord/types/allowed_mentions.py Outdated Show resolved Hide resolved
nextcord/types/channel.py Outdated Show resolved Hide resolved
nextcord/types/embed.py Show resolved Hide resolved
nextcord/types/embed.py Show resolved Hide resolved
nextcord/types/protocols/abc.py Outdated Show resolved Hide resolved
nextcord/types/protocols/abc.py Outdated Show resolved Hide resolved
nextcord/types/protocols/base_user.py Outdated Show resolved Hide resolved
nextcord/types/protocols/base_user.py Show resolved Hide resolved
nextcord/types/snowflake.py Outdated Show resolved Hide resolved
nextcord/types/user.py Outdated Show resolved Hide resolved
@ooliver1
Copy link
Member

for your class methods, doesnt cls() do the same as using .__init__()

@Gugu7264
Copy link
Author

for your class methods, doesnt cls() do the same as using .__init__()

Yeah in that case it does, so imma change it but the embed API will need changes anyways, still unsure how/what to change

@Gugu7264 Gugu7264 requested review from TAG-Epic and ooliver1 January 29, 2022 18:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants