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 back the channel topic #1234

Closed
shalkya opened this issue May 7, 2019 · 3 comments · Fixed by #1335
Closed

add back the channel topic #1234

shalkya opened this issue May 7, 2019 · 3 comments · Fixed by #1335
Milestone

Comments

@shalkya
Copy link
Member

shalkya commented May 7, 2019

so TD can actually ping stuff in the channel (eg : challonge link, streams etc)

@Geosearchef
Copy link
Member

issue 1234, wow

Probably not gonna happen, there was a lot of debate about this (don't remember where, may have been slack) and the common opinion was that the channel topic never was the solution needed. It was removed from the java client on purpose.

@Brutus5000
Copy link
Member

The latest info from Slack was Downlord would be okay with it if we add it as sort-of-message on opening the channel. So it's not supposed to stay non-stop visible, but only if a channel is opened or the topic is changed.

@1-alex98
Copy link
Member

1-alex98 commented Jul 2, 2019

Moderators are really keen of having it visible non-stop...

1-alex98 added a commit that referenced this issue Jul 2, 2019
1-alex98 added a commit that referenced this issue Jul 2, 2019
1-alex98 added a commit that referenced this issue Jul 2, 2019
1-alex98 added a commit that referenced this issue Jul 4, 2019
1-alex98 added a commit that referenced this issue Jul 22, 2019
1-alex98 added a commit that referenced this issue Jul 22, 2019
1-alex98 added a commit that referenced this issue Jul 22, 2019
Add channel topics

Fix #1234
@1-alex98 1-alex98 added this to the v1.0.0 milestone Aug 29, 2019
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 a pull request may close this issue.

5 participants