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

Reply with source if users ask for it #5

Merged
merged 17 commits into from
Jan 30, 2019
Merged

Conversation

dergigi
Copy link
Owner

@dergigi dergigi commented Jan 30, 2019

Resolves issue #1.

I introduced a new script which can be executed with yarn reply. The bot will search twitter for unanswered replies which contain the word "source". If there are any, the bot will perform a lookup of the quote source and post a reply in the following form:

Satoshi wrote this on November 11, 2008, in an email to the cryptography mailing list. You can find the full quote, context, and more information here: https://satoshi.nakamotoinstitute.org/emails/cryptography/10/

Added some tests for the lookup and some other things. There could be more thorough testing, maybe I'll get around to write tests for every quote.

I plan to set up a job to execute the reply script every 10 minutes.

@dergigi dergigi self-assigned this Jan 30, 2019
@dergigi
Copy link
Owner Author

dergigi commented Jan 30, 2019

Oh, I also had to move the actual posting of a random quote out of the "root" of the bot, otherwise, it would post a random quote every time it tries to reply to users (every 10 minutes). See commit
8724f67

@dergigi
Copy link
Owner Author

dergigi commented Jan 30, 2019

Since there is no CI yet (see #4) I will merge this now and see how it goes in the wild.

test and lint do pass:

screen shot 2019-01-29 at 22 27 52

screen shot 2019-01-29 at 22 27 40

@dergigi dergigi merged commit aae4052 into master Jan 30, 2019
@ghost
Copy link

ghost commented Jun 28, 2022

🍀

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.

1 participant