Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

Add story command #42

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

Add story command #42

wants to merge 12 commits into from

Conversation

tmttn
Copy link
Contributor

@tmttn tmttn commented Oct 4, 2020

This PR add the boo!story command. It will pull a semi-random story off reddit.com/r/scarystories and post it in the channel where the command was executed.

Time to get your storytelling going!

This is how it works:
boostory

Caveats:

@tmttn tmttn changed the title 👄 Add story command Add story command Oct 4, 2020
@illusi0nary illusi0nary added enhancement New feature or request hacktoberfest Issue dedicated to the Hacktoberfest labels Oct 4, 2020
@ALMerrill
Copy link
Contributor

A couple things I just wanted to mention, taking a look at this.
First, we should try to get the anti-spam feature in before this, so that we don't get anyone spamming it and super filling the channel with stories. Or even if its not a per-user anti-spam, it could be a general story command anti-spam or something.
Second, I saw that it adds some setup for local testing. Could we maybe make the reddit setup optional for local testing? That way if we're working on something unrelated to the story command, we can skip those steps and that command just won't work, maybe sending a message saying its not configured. Hopefully that makes sense. Basically I just want to avoid having people getting reddit stuff setup when they won't even be using the command. We can move those local testing setup instructions to their own section, saying to follow them if you're working on the story command.
Happy to hear your thoughts on those!

@tmttn
Copy link
Contributor Author

tmttn commented Oct 5, 2020

@ALMerrill I made the config optional. I changed the story to a DM for now. We can re-evaluate where the story is posted after #35 is implemented.

@ALMerrill
Copy link
Contributor

Ok cool. Yeah I think it would be awesome if the stories showed up in the channel, as long as there’s a time limit before allowing it again haha. As a quick fix, you could probably just set a datetime variable whenever story is called, and then check that variable when it is called again, and if it’s been a certain amount of time, maybe a minute or something, then it goes, otherwise it DMs them and says it hasn’t been long enough.
Sorry I don’t mean to take a control lol, I’m not even the maintainer. Just figured I’d share my thoughts, and spamming is on my mind

@tmttn
Copy link
Contributor Author

tmttn commented Oct 5, 2020

No worries it's all for the greater good. 🗡️

I'm working on the temporary spam protection for this command.

@LucasCtrl LucasCtrl added this to the Release v1.3.0 milestone Oct 6, 2020
@illusi0nary
Copy link
Contributor

We have merge conflicts here because of the docker deployment.

Please use git rebase main or git merge main on your branch in order to make it mergeable :-)

@illusi0nary illusi0nary self-requested a review October 12, 2020 17:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request hacktoberfest Issue dedicated to the Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants