FriendsOfFlarum Merge Discussions

datitisev

Merge Discussions by FriendsOfFlarum

Merge two or more discussions into one.

Installation

Use Bazaar or install manually with composer:

composer require fof/merge-discussions

Updating

composer update fof/merge-discussions

Links

An extension by FriendsOfFlarum, commissioned by giffgaff.

matteocontrini

Nice!

It would be great to also have a similar (?) extension for moving specific posts to an existing discussion. The modal component can be reused, probably.

luceos

matteocontrini lovely idea, this one was commissioned though, so the requirements were set.

datitisev

matteocontrini That sounds more like flagrow/split instead of fof/merge-discussions... though you could combine the two and split those posts, then merge them (the whole new discussion) into another discussion.

matteocontrini

datitisev yeah of course, it wasn't a feature request for this extension, but the modal thing is something that would be required to choose the destination discussion. The combination of the two is something I did not think about, but it's still a bit unnatural and probably confusing for users...

Anyway, a question about this extension: when merging two discussions, is the chronological order of the posts preserved, so that they are inserted at the right place between existing posts?

Thanks!

luceos

matteocontrini Anyway, a question about this extension: when merging two discussions, is the chronological order of the posts preserved, so that they are inserted at the right place between existing posts?

Yes. Both discussions are merged together, posts are renumbered based on post creation time, this includes the OP.

luceos

Let me clarify that even though the initial version was commissioned we'd love to see where we get with this extension based on your input, experiences and use cases 💝

datitisev

0.1.1

Fixed incorrect post numbering on merge post event (https://github.com/FriendsOfFlarum/merge-discussions/issues/1)

Updating

composer update fof/merge-discussions

datitisev

0.2.0

Added button to preview discussion after all discussions have been merged
Added help text to avoid confusion as to where the merged discussions will be merged to

Updating

composer update fof/merge-discussions

datitisev

0.2.1

Fix discussion data relating to participants and posts not being updated
- https://github.com/FriendsOfFlarum/merge-discussions/issues/4

Updating

composer update fof/merge-discussions

datitisev

0.2.2

Allow searching discussions by ID
- bypasses search issues on large forums such as this one

Updating

composer update fof/merge-discussions

datitisev

0.3.1

Add option to merge current discussion into another
- Before 0.3.0, you could only merge other discussions into the current one
Clarify wording of what will be merged where
Add discussion title to the preview so it's clearer which discussion the other discussion(s) are being merged into
0.3.1 Fix width of the preview hero

Updating

composer require fof/merge-discussions

luceos

datitisev great job David 🤘