mvbot icon indicating copy to clipboard operation
mvbot copied to clipboard

Mass Moves

Open LazerRhome opened this issue 4 years ago • 21 comments

Just seeing if it's possible to make it so that we can move multiple messages at once - maybe if we make it so you can post multiple IDs separated by a comma? (don't know if that's possible as I'm not a coder!)

Thank you!

LazerRhome avatar Jan 06 '21 18:01 LazerRhome

Just seeing if it's possible to make it so that we can move multiple messages at once - maybe if we make it so you can post multiple IDs separated by a comma? (don't know if that's possible as I'm not a coder!)

Thank you!

@tmanifold I'm really happy with the bot, was a headache bothering me for some time that i couldn't move messages between channels. BUT: I would ADORE the developer if batch- moving messages could be enabled! Other than that, happy new year and keep up the good work!

WeserBlitz avatar Jan 07 '21 16:01 WeserBlitz

Thanks for the feedback! I will look into this when I have some extra time.

tmanifold avatar Jan 07 '21 18:01 tmanifold

Thanks for the feedback! I will look into this when I have some extra time.

You rock! Thank you a million!!! Hero!

WeserBlitz avatar Jan 07 '21 20:01 WeserBlitz

I too would love some batch movement capabilities!! :D

Maybe allowing something like this with comma separation: !mv <message-id,message-id,message-id,message-id,message-id> <target-channel> ["reason"]

Or maybe somehow select all messages from last (15 minutes) -for when people go off on a tangent- to a new channel: !mv <all> <15> <target-channel> ["reason"]

And of course, hopefully they would be in the same order as before the move :D

Thanks for your hard work!

videovillain avatar Mar 28 '21 08:03 videovillain

Due to all the interest this feature is actually in the works. I recently did rewrite of large portions of the code to allow for this functionality and am playing with some different methods of implementation. This should be live in some form or another in the next update.

tmanifold avatar Mar 28 '21 15:03 tmanifold

Due to all the interest this feature is actually in the works. I recently did rewrite of large portions of the code to allow for this functionality and am playing with some different methods of implementation. This should be live in some form or another in the next update.

This is great news! @tmanifold you're awesome <3 Thanks!

WeserBlitz avatar Mar 28 '21 17:03 WeserBlitz

The new version will make use of command-line style switches. Currently, I've implemented a switch to move the next n number of messages in addition to the specified message. I am thinking about ways to implement a delimited list. I also like @videovillain's idea of moving messages within the last x amount of time, so i will probably work on this too.

An example of the new syntax might be: !mv -m <message-id> -d <destination-channel> [-c "comment"] [-n <number-of-messages]

I'm planning to make the default behavior for bulk moves to only write the mvbot header (the part containing the original user, channel, time, and comment) for the first moved message for each user in a span of messages: image

I am planning to make delimited lists an exception to this rule; they will have a header for every message. This can be overridden with another planned switch, ex --no-header.

Additionally, if i can maintain backwards compatibility with the current command syntax then I will. However it is likely to be deprecated in future versions.

The current development build is on the dev branch, and implements the -n switch.

I appreciate everyone's feedback and input. This type of engagement helps make mvbot better!

tmanifold avatar Mar 28 '21 18:03 tmanifold

So the no-header option doesn't exist for the current master, is that correct? Is there a rough timeline for the next update, or should I shut up and sit down to wait patiently. :)

changeling avatar Apr 09 '21 05:04 changeling

Will the next update allow moving a message cleanly, so that it appears just as it did before the move, if the no-headers flag is set? That's really what I'm looking for, those this is sooo close. :)

changeling avatar Apr 09 '21 06:04 changeling

@changeling correct, these features are not yet live. My plan for something like a --no-header flag is to simply have the bot move the message content and not post extra info about who or when the original message occurred. This will likely be detailed greater on the dev branch when I get a more functional build ready.

As for a timeline on the next update, I don't have much of an estimate. I am working on my final year of university, and work on the bot when time and energy allows.

You patience is appreciated, big changes are to come and I think they will be good for the functionality and utility of mvbot.

tmanifold avatar Apr 09 '21 19:04 tmanifold

Thanks for all your work on this! Absolutely patient on my end.

Just for clarity, will the moved messages appear just as it did in the original channel, eventually? Date, time, poster? Ideally we’d like to have position and threading appear the same in the new location.

And good travels with university!

Chris

On Fri, Apr 9, 2021 at 2:49 PM Tyler M @.***> wrote:

@changeling https://github.com/changeling correct, these features are not yet live. My plan for something like a --no-header flag is to simply have the bot move the message content and not post extra info about who or when the original message occurred. This will likely be detailed greater on the dev branch when I get a more functional build ready.

As for a timeline on the next update, I don't have much of an estimate. I am working on my final year of university, and work on the bot when time and energy allows.

You patience is appreciated, big changes are to come and I think they will be good for the functionality and utility of mvbot.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/tmanifold/mvbot/issues/4#issuecomment-816929899, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADB37QSI72FJ5A44TFJWJTTH5K43ANCNFSM4VX4RQOQ .

changeling avatar Apr 09 '21 19:04 changeling

Just for clarity, will the moved messages appear just as it did in the original channel, eventually? Date, time, poster?

Unfortunately this isn't possible due to the limitations of the Discord API. The mvbot header is a stand-in for this.

tmanifold avatar Apr 10 '21 00:04 tmanifold

One idea I had would be to use webhooks to post the messages with a custom name and avatar. However, a webhook would need to be made for every channel.

jerbear2008 avatar Apr 24 '21 15:04 jerbear2008

One idea I had would be to use webhooks to post the messages with a custom name and avatar. However, a webhook would need to be made for every channel.

The way I understand webhooks, there isn't really a feasible way to implement this. Unless the discord API significantly changes, the bot copying message content and referencing the original user is likely the best that can be done.

tmanifold avatar Apr 24 '21 15:04 tmanifold

Just for clarity, will the moved messages appear just as it did in the original channel, eventually? Date, time, poster?

Unfortunately this isn't possible due to the limitations of the Discord API. The mvbot header is a stand-in for this.

Gotcha! Sorry for the delayed response.

On that note, though, will the resulting messages sans header at least look "clean?" That is, will they simply be the content of the message?

And the obligatory, no pressure just asking, question:

Any idea of timeframe this might commit?

changeling avatar Apr 28 '21 16:04 changeling

Yeah, it would do that. However, you would miss timestamps and where the message originally came from.

jerbear2008 avatar Apr 28 '21 21:04 jerbear2008

And the obligatory, no pressure just asking, question:

Any idea of timeframe this might commit?

@changeling, please refer to my earlier reply

tmanifold avatar Apr 30 '21 02:04 tmanifold

And the obligatory, no pressure just asking, question: Any idea of timeframe this might commit?

@changeling, please refer to my earlier reply

Which I should have done before. Thanks for the pointer, with apologies.

changeling avatar Apr 30 '21 05:04 changeling

I'm gonna be honest, I love this bot, and I wish I could implement this command into my own bot, but I suck at coding, since I just started out a couple months ago. I tried re-coding it to work within an "advanced" command handler, to make it work in a bot with multiple commands, using the multiple file separation method, and was stumped within 20 minutes.

zkll505 avatar May 20 '21 02:05 zkll505

I'm gonna be honest, I love this bot, and I wish I could implement this command into my own bot, but I suck at coding, since I just started out a couple months ago. I tried re-coding it to work within an "advanced" command handler, to make it work in a bot with multiple commands, using the multiple file separation method, and was stumped within 20 minutes.

Glad you like the bot! I have a working version of this in the current dev branch. Just a few things to iron out when I have time and hopefully it will be ready to push to main. As for the coding, don't give up! Just like any other skill, it just takes time and practice.

tmanifold avatar May 20 '21 03:05 tmanifold

Excited to see this implemented! I'm currently trying to combine a few channels and this would make it so much easier! Thanks for doing what Discord has refused to do for years. lol

DorkyDino avatar Aug 23 '21 19:08 DorkyDino