Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File separation #99

Closed
wants to merge 47 commits into from
Closed

File separation #99

wants to merge 47 commits into from

Conversation

Jerry-py
Copy link
Contributor

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
  • - 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

@Jerry-py
Copy link
Contributor Author

I haven't checked the code if it has worked or not.

@Jerry-py
Copy link
Contributor Author

Some are blank for some reason idk why

@Jerry-py Jerry-py marked this pull request as draft March 10, 2022 20:44
@Jerry-py Jerry-py marked this pull request as ready for review March 10, 2022 20:51
@Jerry-py
Copy link
Contributor Author

Finished adding the code to the blank files

@Jerry-py
Copy link
Contributor Author

Sorry for the all commits doing this in the browser

@Jerry-py
Copy link
Contributor Author

@TheUntraceable Done

Copy link
Member

@TheUntraceable TheUntraceable left a comment

Choose a reason for hiding this comment

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

I don't mean to be really really mean and want it really really one way, but Integrations are part of guilds.py, could you move it there please?

@Jerry-py
Copy link
Contributor Author

Alr

Copy link
Contributor Author

@Jerry-py Jerry-py left a comment

Choose a reason for hiding this comment

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

Done

Copy link
Member

@nav-github01001 nav-github01001 left a comment

Choose a reason for hiding this comment

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

@Jerry-py Can you look at my fork of this pr and try to apply changes in it?
https://github.com/nav-github01001/EpikCord.py/tree/patch-8

EpikCord/guild.py Outdated Show resolved Hide resolved
@Jerry-py
Copy link
Contributor Author

@TheUntraceable Done

@@ -210,6 +210,36 @@ def __init__(self, client, data: dict):
self.channel_id: str = data.get("channel_id")
self.privacy_level: int = data.get("privacy_level")
self.discoverable_disabled: bool = data.get("discoverable_disabled")
class TextBasedChannel:
Copy link
Member

Choose a reason for hiding this comment

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

This is meant to be client.utils.channel_from_type(dict)

@Jerry-py
Copy link
Contributor Author

We have conflicts idk how to resolve

@Jerry-py
Copy link
Contributor Author

So i wanna say we have to close this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants