sir-lancebot icon indicating copy to clipboard operation
sir-lancebot copied to clipboard

A Bookmark Command

Open NeilShah2026 opened this issue 4 years ago • 10 comments

Description

When a user tends to use a command often, either by the user manually adding it, or have the bot automatically add through command tracking, add a ".bookmarks", and it will pull up all the bookmarked commands of that user.

Reasoning

For easier use and memorization of the bot commands.

  • [ ] I'd like to implement this feature myself
  • [x] Anyone can implement this feature

NeilShah2026 avatar Mar 26 '21 19:03 NeilShah2026

@NeilShah2026 This bot currently doesn't have a database, such as postres.

I don't think many users would like to be message automatically if the bot noticed a pattern that is set by some constant.

The bot does have a feature for bookmarking a message, so if the user wants to save something for later, they can always do so.

Xithrius avatar May 29 '21 05:05 Xithrius

A possible way for doing this is by going through each message in the user DM history and checking if the message is an embed and if so it has the title of Bookmark.

Shivansh-007 avatar Jun 08 '21 04:06 Shivansh-007

That's not the best idea if the user has a long message history with the bot.

Xithrius avatar Jun 08 '21 16:06 Xithrius

Agreed, a better way I just remembered as we had discussed in dev-contrib was to pin the bookmark messages in DMs of the user and then the bot could go through the pin history?

Shivansh-007 avatar Jun 08 '21 16:06 Shivansh-007

True, that would also work

NeilShah2026 avatar Jun 08 '21 17:06 NeilShah2026

I don't think many users would like to be message automatically if the bot noticed a pattern that is set by some constant.

I still think this is relevant. Not sure if the default should have this feature turned on or off.

Xithrius avatar Jun 12 '21 20:06 Xithrius

I don't want to fill up the user's pinned messages, since it may collide with something in the future.

If anything, we should go with manual additions instead of automatic bookmarking so users aren't randomly DM'd.

Xithrius avatar Sep 06 '21 19:09 Xithrius

@ NeilShah2026 This bot currently doesn't have a database, such as postres.

To note, at this point, the bot basically has a database with redis.

onerandomusername avatar Sep 06 '21 19:09 onerandomusername

Well, yes, but I'm not sure if it's suited for long-term storage. We'd have to put some time limit on the bookmark or something so we don't use up storage for no reason.

Xithrius avatar Sep 06 '21 19:09 Xithrius

Wait what is this issue actually about? The original issue describes the ability to bookmark commands for ease of finding again, but then the comments start talking about our message bookmarking command. Can this be clarified?

ChrisLovering avatar Oct 21 '21 17:10 ChrisLovering