search

VTGare / boe-tea-go

Source image bot for Discord
MIT License
20 stars 7 forks source link

Handle multiple images with SauceNAO command #28

Closed XangelMusic closed 1 year ago

XangelMusic commented 1 year ago

func sauce() func findImages()

added func sauceNAOMultipleEmbed()

This is have Boe Tea deal with messages with multiple images with bt!sauce, and post an embed with several SauceNAO links for each image. It should not change the way Boe Tea handles messages with one image.

VTGare commented 1 year ago

Not sure about the value of this change. In 99% of cases people want to use the command on just one image and then for the 1% that want multiple images we're just showing them direct links to SauceNAO, which doesn't seem useful to me.

XangelMusic commented 1 year ago

It would provide an avenue for users as not many people know about SauceNAO or know how to sauce images. I'm also hoping to have Boe Tea react to the images posted in my server without any sauce using this change I made.

That said, I actually saw a suggestion for an app context menu command as well. It will certainly save chat space instead.

VTGare commented 1 year ago

I actually saw a suggestion for an app context menu command as well.

That's definitely something we should add eventually.

As for this change, sorry I think it's not going to be very useful in the main bot. Maybe I'll come back to this idea in the future when SauceNAO will let us do batch requests.