Skip to content

Add New Voice Channels To Config#1351

Closed
HassanAbouelela wants to merge 1 commit into
python-discord:masterfrom
HassanAbouelela:add_voice_channel
Closed

Add New Voice Channels To Config#1351
HassanAbouelela wants to merge 1 commit into
python-discord:masterfrom
HassanAbouelela:add_voice_channel

Conversation

@HassanAbouelela
Copy link
Copy Markdown
Contributor

Adds a new general voice and text channel to the default config, and renames general and help channels to match the new configuration in the config file and constants file. Announcment with details.

Adds the new voice and text channels to the default config, and renames
all affected channels in the config and constants to match the new
names.
@HassanAbouelela HassanAbouelela added the a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) label Jan 15, 2021
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.006%) to 57.357% when pulling 138f9f2 on HassanAbouelela:add_voice_channel into 8b1276a on python-discord:master.

@MarkKoz
Copy link
Copy Markdown
Contributor

MarkKoz commented Jan 19, 2021

Can you elaborate on why these are being added? Are they required by #1297? If so, it isn't necessary to open a separate PR.

@HassanAbouelela
Copy link
Copy Markdown
Contributor Author

I originally asked to update it for that PR, but was then asked to update in general (conversation). If you think it should be added on that PR, feel free to close this one.

@MarkKoz
Copy link
Copy Markdown
Contributor

MarkKoz commented Jan 20, 2021

I believe what was told to you is false; nothing else relies on these constants. I think it makes more sense to make these changes in #1297.

@MarkKoz MarkKoz closed this Jan 20, 2021
@HassanAbouelela HassanAbouelela deleted the add_voice_channel branch January 20, 2021 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants