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

boobs.lua: accept parameters #276

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

boobs.lua: accept parameters #276

wants to merge 10 commits into from

Conversation

ruvolof
Copy link

@ruvolof ruvolof commented Jun 25, 2015

Added optional parameter for boobs and butts to send more pictures.

@ruvolof
Copy link
Author

ruvolof commented Jun 27, 2015

Added extra commit to fix danbooru plugin. It was sending the link to the image, telegram was showing the picture and the the images plugin was resending the image. Now it sends the same information (without the link) and the picture is sent after as the boobs plugin is doing.

@yagop
Copy link
Owner

yagop commented Jun 27, 2015

When sending multiple images, it should be done using callbacks.
A maximum number of images must be set in order users don't flood the bot.

@@ -38,21 +38,28 @@ local function getRandomBoobs(attempt)
end

local function run(msg, matches)
local url = nil
n = string.match(matches[1], '[0-9]+')
Copy link
Owner

Choose a reason for hiding this comment

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

This should be local

@ruvolof
Copy link
Author

ruvolof commented Jul 15, 2015

Added the limit. Not really clear what you mean about callbacks request.

@pcortizona
Copy link

Testing....

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.

3 participants