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

Remove global chat integration. #14496

Merged
merged 1 commit into from Dec 13, 2017

Conversation

Projects
None yet
6 participants
@pchote
Member

pchote commented Dec 11, 2017

A lot has been said and done over the years to try and encourage the global chat as a community service, but the facts are that it remains mostly unused and most of the use it does get is counter to our community code of conduct and basic respect.

The vast majority of messages in the channel are one of:

  • Spamming ]games
  • Insulting players and/or developers
  • Spamming jibberish to make orabot kick themselves

There are occasional support requests (usually how to handle port forwarding), but these often go unanswered. There are usually a few "normal" messages each day ("join server x", etc), but this usage really can't justify keeping what has become a hub of antisocial and unwanted (in at least my opinion) behaviour. There has been little enthusiasm about trying to moderate the channel, and I do not believe that to be worth the effort even if we could find enough volunteers.

This PR removes all global chat integration. If it is merged then I would like to think that somebody can work on a couple of followups to help address the other usecases of the global chat:

  • Exposing the vestigial client list that is included in the master server data in a bin below the map preview in the server browser (the full plan of exposing spawn points and teams is be too much).
  • Exposing the server list in some form as a tab from the MP game lobby, like global chat is now.

@pchote pchote added this to the Next release milestone Dec 11, 2017

@abcdefg30

😢

@CH4Code

This comment has been minimized.

Show comment
Hide comment
@CH4Code

CH4Code Dec 12, 2017

Contributor

Damn, probably 50% of fresh recruits for d2k Thursday came from that :/

Contributor

CH4Code commented Dec 12, 2017

Damn, probably 50% of fresh recruits for d2k Thursday came from that :/

@Arular101

This comment has been minimized.

Show comment
Hide comment
@Arular101

Arular101 Dec 12, 2017

Contributor

I think it's fine to remove this feature. I've seen some other games that removed this too because of abuse in different ways.

Damn, probably 50% of fresh recruits for d2k Thursday came from that :/

Maybe you can use the OpenRA Facebook or Reddit to get more people.

Contributor

Arular101 commented Dec 12, 2017

I think it's fine to remove this feature. I've seen some other games that removed this too because of abuse in different ways.

Damn, probably 50% of fresh recruits for d2k Thursday came from that :/

Maybe you can use the OpenRA Facebook or Reddit to get more people.

@pchote

This comment has been minimized.

Show comment
Hide comment
@pchote

pchote Dec 12, 2017

Member

Maybe you can use the OpenRA Facebook or Reddit to get more people.

D2K Thursday and similar events are definitely the kind of thing we should be advertising more on social media.

Member

pchote commented Dec 12, 2017

Maybe you can use the OpenRA Facebook or Reddit to get more people.

D2K Thursday and similar events are definitely the kind of thing we should be advertising more on social media.

@penev92

There is still a mention of SmartIrc4Net in AUTHORS

@Mailaender

This comment has been minimized.

Show comment
Hide comment
@Mailaender

Mailaender Dec 13, 2017

Member

👍 though I am frustrated as well if we count the man hours @ScottNZ, myself and countless other contributors put into the feature. From a technical standpoint we also have to get rid of the outdated NuGet dependency meebey/SmartIrc4net#33 anyway.

Member

Mailaender commented Dec 13, 2017

👍 though I am frustrated as well if we count the man hours @ScottNZ, myself and countless other contributors put into the feature. From a technical standpoint we also have to get rid of the outdated NuGet dependency meebey/SmartIrc4net#33 anyway.

@pchote

This comment has been minimized.

Show comment
Hide comment
@pchote

pchote Dec 13, 2017

Member

Updated.

Member

pchote commented Dec 13, 2017

Updated.

@Mailaender Mailaender merged commit 502c3e2 into OpenRA:bleed Dec 13, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@penev92

This comment has been minimized.

Show comment
Hide comment
@penev92
Member

penev92 commented Dec 13, 2017

@Mailaender

This comment has been minimized.

Show comment
Hide comment
@Mailaender

Mailaender Dec 21, 2017

Member

D2K Thursday and similar events are definitely the kind of thing we should be advertising more on social media.

Done.

Member

Mailaender commented Dec 21, 2017

D2K Thursday and similar events are definitely the kind of thing we should be advertising more on social media.

Done.

@pchote

This comment has been minimized.

Show comment
Hide comment
@pchote

pchote Dec 29, 2017

Member

Followup PRs to restore the useful functionality from the chat:

Member

pchote commented Dec 29, 2017

Followup PRs to restore the useful functionality from the chat:

@pchote pchote deleted the pchote:remove-global-chat branch Apr 28, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment