do add permissions group allow to give access for certain users based on roles or group like we have the similar feature in merge & split extensions

Example

could you please increase the space of this window so we can use the search more efficiently
currently, we could not able to read the full title of the discussion when we are searching

thanks in advance.

Once we hit move post from the 3dot section it would be good if all posts get a checkbox where we can easily tick the posts and add them to our cart

it's a bit hard to select each and every post when we want to move 4 to 8 posts with the current option we have.

Thanks in advance

Steps to reproduce:

1. Find discussion with only one post.
2. Add or remove tags, so DiscussionTaggedPost posts is created.
3. Move first post to different discussion.

Result:

Sub does not inherit post number from moved post and it is displayed after change tag event:

It looks like post number is overwrite in before create hook:

https://github.com/flarum/framework/blob/6de8113720408127bc9fc98339448069ee956e35/framework/core/src/Post/Post.php#L99-L105

unable to move posts between discussions when their content is the same or posts are created at the same time.

and also I've noticed being unable to move the same posts back and forth multiple times within the same discussions.

not sure which scenarios are causing the error Oops! Something went wrong. Please reload the page and try again.

I was testing how it would act in such scenarios, not intended to break things on purpose 😅

when I move the first post the discussion is getting locked but an activity event is not placed as "the discussion is locked"

It would be nice if we Notify post owner as post is moved to different discussion (web notification only) so he can easily go through it.

If user clicks on notification take him to new moved discussion

I am not sure about this but split or merge extensions have this feature. (Quoting those extensions because you may find existing code examples to use for our need 🙂)

It might sound stupid, but I just did it. :P Discussion had only one posts and I moved first post to the same discussion (I pasted discussion ID in destination discussion input). Result look like this:

It would be great to have some protection against such mistakes, since it should be relatively simple to implement and such operation never makes sense.

[2023-12-02 03:12:29] flarum.ERROR: TypeError: SychO\MovePosts\Formatter\UnparsePostMentions::__invoke(): Argument #2 ($xml) must be of type string, null given, called in /public_html/vendor/flarum/core/src/Foundation/ContainerUtil.php on line 30 and defined in /public_html/vendor/sycho/flarum-move-posts/src/Formatter/UnparsePostMentions.php:18

Add: protected $xml;

When the poster has deleted their account, an error is generated in http request and the button continues to spin:

It's related to this line:

MovePostsHandler.php

/**
 * Creates a target discussion when specified.
 */
protected function createTargetDiscussion(Discussion $sourceDiscussion, CommentPost $firstPost, string $title, bool $emulate): Discussion
{
    $discussion = Discussion::start($title, $firstPost->user);

When starting the discussion it appears an actual User is required, but the user of the post deleted their account. Maybe it needs to support Guest user of some kind here?

hello, after the update to flarum v1.3 this extension is not working, the screen is getting stuck at proceed step and the loading icon is continue to load forever. I request you to please look into this.

you can also test this at https://nightly.flarum.site/d/42-sdfds

Flarum 1.3.0
PHP 7.4.27
MySQL 5.7.34-log

Like we have a search feature while merging discussion into an existing discussion in merge extension consider adding the similar feature in move posts extensions.

it will be very much helpful to search existing posts based on discussion ID or discussion title

Preview is not needed if it's possible and easy to implement display preview of above and below 1-2 posts like merge extension does.