FriendsOfFlarum Split, separates posts to a new discussion

tjrgg

Pollux You're right, I misread it. My bad.

lisk

I really like these two extensions that you created Split and Byobu

lisk

Found a bug in beta 7
split does not work for moderators and even for regular users.
You do not have permission to do that.

Victor12

lisk Make sure Moderators have the permission to do that in your Permissions (/admin#/permissions) section. Could be set to Admins by default.

lisk

Victor12 This I did long ago

lisk

Does everyone have this mistake or just me?

jordanjay29

lisk It's not just you: flagrow/split14

Jasper

I am very grateful to the developers for their efforts. I have a question. Is the segmentation operation reversible?

luceos

Jasper No, there's no merge (or undo split) option yet. Having a merge extension makes sense though.

Jasper

luceos thank you _:: 😀

luceos

Tagged 0.3.2 which fixes the permission problem 😊 That it even existed is a huge embarrassment, my apologies.

lisk

luceos can you help me if it's not difficult?
Tell me where to add this part of the code to combine Split and Byobu
https://github.com/flagrow/byobu/blob/0.1.2/js/forum/dist/extension.js#L18-L20
I want to Split asked who to add to the discussion as Byobu

MikeJones

when splitting can i pick individual posts to send with it?

luceos

MikeJones no only a range for now

MikeJones

luceos Got it. Just installed it and it worked perfectly for my use case. But yea picking individual posts may be useful in the future!

luceos

MikeJones yes this is just the initial version based on a very pressing issue here on discuss.

luceos

Released 0.4 which fixes a bug that disables loading of any successive extension. The extension is moved to the Friends of Flarum team as well. Luckily this doesn't really have an impact because the permission wasn't prefixed and there are not database migrations; upgrading still requires you to delete flagrow/split first:

$ composer remove flagrow/split
$ composer require fof/split

matteocontrini

luceos what was it, specifically? I can't find the fix in recent commits 🤔

MikeJones

luceos just updated thanks!

luceos

Here's the PR by @Franz back then: https://github.com/FriendsOfFlarum/split/pull/19