Navigation Menu

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

Add support for discord category channels (discord) #863

Merged
merged 1 commit into from Jul 15, 2019
Merged

Add support for discord category channels (discord) #863

merged 1 commit into from Jul 15, 2019

Conversation

42wim
Copy link
Owner

@42wim 42wim commented Jul 14, 2019

This adds support for the discord category option that can be used
to group channels in. This means we can have multiple channels with
the same name.

We add the option to specify a category in the channel option of a
discord account under [[gateway]]

Besides channel="channel" or channel="ID:channelID", now also
channel="category/channel" can be specified.

This change remains backwards compatible with people that haven't
specified the category and incorporates the fix in #861

@qaisjp sorry but this roll backs a bit of your code in #861

Any comments?

This adds support for the discord category option that can be used
to group channels in. This means we can have multiple channels with
the same name.

We add the option to specify a category in the channel option of a
discord account under [[gateway]]

Besides channel="channel" or channel="ID:channelID", now also
channel="category/channel" can be specified.

This change remains backwards compatible with people that haven't
specified the category and incorporates the fix in #861
@42wim 42wim added this to the 1.15.1 milestone Jul 14, 2019
@codeclimate
Copy link

codeclimate bot commented Jul 14, 2019

Code Climate has analyzed commit 67107da and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 0.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 9.8% (0.0% change).

View more on Code Climate.

@qaisjp
Copy link
Collaborator

qaisjp commented Jul 15, 2019

Is this the line that incorporates the fix in #861?

Much better than my solution!

Slash is an invalid character for a channel name on Discord, so this seems fine.

@42wim
Copy link
Owner Author

42wim commented Jul 15, 2019

Is this the line that incorporates the fix in #861?

Yup

Slash is an invalid character for a channel name on Discord, so this seems fine.

Ah good point about the slash, didn't think about that :)

@42wim 42wim merged commit 7d2e440 into master Jul 15, 2019
@42wim 42wim deleted the f-dcat branch August 17, 2019 21:53
zeridon pushed a commit to zeridon/matterbridge that referenced this pull request Feb 12, 2020
This adds support for the discord category option that can be used
to group channels in. This means we can have multiple channels with
the same name.

We add the option to specify a category in the channel option of a
discord account under [[gateway]]

Besides channel="channel" or channel="ID:channelID", now also
channel="category/channel" can be specified.

This change remains backwards compatible with people that haven't
specified the category and incorporates the fix in 42wim#861
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

2 participants