search

Marenthyu / NepBot

Waifu Trading Card Game Bot for Twitch
https://waifus.de/help
BSD 3-Clause "New" or "Revised" License
4 stars 6 forks source link

Added command for the nepdoc link #88

Closed zfbx closed 4 years ago

zfbx commented 4 years ago

A lot of people prefer this help document and like to have it on hand.

Marenthyu commented 4 years ago

We tried to stay somewhat independant of it, but i guess there's no real harm in embracing it further.

I'd refer the command to respond with a full sentence instead of just the link though.

Otherwise LGTM

Marenthyu commented 4 years ago

Any comments @xorhash ?

xorhash commented 4 years ago

!secretdoc used to exist. I see no harm in having the command.

I'd [p]refer the command to respond with a full sentence instead of just the link though.

I don't really see much point in that; the only people who'd try to use it either probably already know what it is.

I'd suggest adding a new config variable to read the URL from. I occasionally move the thing around (happened at least twice in 1.5 years); having to restart the bot and push a code update for a minor change like that is pretty brutal.

zfbx commented 4 years ago

How is this @Marenthyu ?

Marenthyu commented 4 years ago

LGTM