Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

Topic split, move and join - problems and solutions #339

Open
Kaik opened this issue Feb 21, 2018 · 0 comments
Open

Topic split, move and join - problems and solutions #339

Kaik opened this issue Feb 21, 2018 · 0 comments

Comments

@Kaik
Copy link
Contributor

Kaik commented Feb 21, 2018

These are problems which I have found during refactoring and solutions that are currently implemented.

Topics join - previously it was appending posts from origin topic to destination one by changing origin posts dates to be one minute older than the last post in destination topic which IMHO is kind of odd but since some might wish this kind of behavior I have added join strategy option where you can choose append or mix - where append works as described while mix just adds posts without changing any dates.

Topics split - this action worked by splitting topic into two and it was based on post ids - which IMHO is wrong since post ids in a topic does not have to be in order (join, move post actions) because posts in a topic are ordered only by post date split is now date based. Because the order in which posts are displayed can be changed by user and splitting can work only one way this might be confusing.

Topic move - works without any problems.

Apart from that, I have added few more options that might be useful. For example, each of those actions on a topic has a reason feature so email can be sent to topic creator - I do consider to add an option to send a reason as well to poster(s) - this could be adjustable (topic creator, posters or both)

Split action has destination forum option for the new topic. Join has an option to leave origin topic as shadow same as shadow topic feature in the move action.

@Kaik Kaik added the Blocker label Feb 21, 2018
@Kaik Kaik added this to the Milestone not yet defined milestone Apr 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant