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.

Sign up for GitHub

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 !crosspost/!crossposting command #32

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add !crosspost/!crossposting command #32

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
d9b5b5d
Add !crosspost/!crossposting command
nickserv Jan 31, 2020
165db50
Change crossposting alias to xpost
nickserv Feb 13, 2020
36984ed
Merge branch 'master' into nickmccurdy-patch-1
nickserv Feb 14, 2020
fa226a7
Merge branch 'master' into nickmccurdy-patch-1
nickserv Mar 5, 2020
c8f805f
Merge branch 'master' into nickmccurdy-patch-1
nickserv Aug 24, 2020
f961fea
Trigger CI build
nickserv Aug 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions features/commands.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -306,6 +306,20 @@ Here's an article explaining the difference between the two: https://goshakkk.na

fetchMsg.delete();
}
},
{
words: [`!crosspost`, `!crossposting`],
help: `explains why and how to avoid crossposting`,
handleMessage: msg => {
msg.channel.send({
embed: {
title: "Crossposting",
type: "rich",
description: `Do not crosspost messages by copying and pasting them into multiple channels. This makes it harder to help you as discussions may be broken up into multiple channels, and takes more space from other users asking questions. Instead, post your message once to the channel you think is most relevant.`,
Copy link
Member

Choose a reason for hiding this comment

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

Maybe soften it a bit, be a little less specific.

Suggested change
description: `Do not crosspost messages by copying and pasting them into multiple channels. This makes it harder to help you as discussions may be broken up into multiple channels, and takes more space from other users asking questions. Instead, post your message once to the channel you think is most relevant.`,
description: `Please don't ask the same question in multiple channels. It makes it harder to help you by splitting the discussion into multiple places, and takes more space from other users asking questions. Instead, post your message once to the channel you think is most relevant.`,

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I prefer having a more generalized definition of crossposting involving posting messages in multiple channels. Asking a question in multiple channels is one form of crossposting, but I'd also like to use this for things like spamming, requests for feedback, notes, and duplicate links.

Copy link
Contributor Author

@nickserv nickserv Aug 24, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

@vcarl Thoughts on this? We can still soften the language but I think it's important to not only recommend against crossposting questions. I updated merge conflicts so otherwise this should be ready to merge.

color: 7506394
}
});
}
}
],
handleMessage: ({ msg, user }) => {
Expand Down