Skip to content
This repository has been archived by the owner on Oct 14, 2018. It is now read-only.

Meme plugin! #180

Merged
merged 2 commits into from Jun 4, 2015
Merged

Meme plugin! #180

merged 2 commits into from Jun 4, 2015

Conversation

rockneurotiko
Copy link
Collaborator

This is the meme plugin with the service of #162
Use disk to store the memes (redis probably is not a good choice here because only one write/read every day) and have a cache of matches.

usage = {
"-----------------------------------------------------------------------------------------------------------------------------",
"Meme commands:",
"-----------------------------------------------------------------------------------------------------------------------------",
Copy link
Contributor

Choose a reason for hiding this comment

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

Won't these make more than one line on mobile devices?
(They look cool on PC anyway)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah! Probably, I get the size from PC XD

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

And yeah, in mobile are 2 lines and a little bit more.
Probably this need to be a little bit less XD

Copy link
Contributor

Choose a reason for hiding this comment

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

maybe the length of what's between them, e.g.

   usage = {
      "--------------",
      "Meme commands:",
      "--------------",

Or ='s:

   usage = {
      "==============",
      "Meme commands:",
      "==============",

Copy link
Owner

Choose a reason for hiding this comment

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

I think those lines aren’t necessary...

@francesco-p
Copy link
Contributor

oh good code! this is the first time I heard about http://hastebin.com/ 👍

@yagop yagop merged commit 567372c into yagop:master Jun 4, 2015
@yagop yagop removed the in progress label Jun 4, 2015
@rockneurotiko rockneurotiko deleted the plugin/meme branch June 21, 2015 20:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants