Merging two posts with a new "merge" feature does not update posts in place as expected

AI-generated summary

A bug has been reported with the “merge” feature, where merging two non-deleted posts does not update the page automatically meglio. Instead, the page only partially updates, removing some messages, but not updating the last message until a page reload meglio. codinghorror initially questioned the use case, but meglio clarified that the issue occurs with non-deleted posts. tgxworld and adrapereira have been notified, and tgxworld has taken a look into the issue, submitting a pull request to resolve it tgxworld.

After merging the two posts as on the following screenshot:

  1. One is marked as removed, which is expected.
  2. The second one is not auto-updated with the new, merged, content. The update can only be seen on page reload.

Hope it hasn’t been reported elsewhere yet.

3 Me gusta

You are merging a deleted post with another post? I am not sure that’s supported.

No, I’m merging two non-deleted posts. The screenshot is the state after
merging.

So what you’re reporting is that everything works fine, the staff member simply has to refresh the page?

Yes. However, it is seen as a bug by staff members, because the page does get updated automatically, but only partially: some of the messages get deleted (highlighted in red immediately), but the other (last) message gets no updated. Thus it looks like you just removed a few messages and this is it… until you refresh the page and realize the messages were merged.

2 Me gusta

@adrapereira Let me know if you know what might be going on? :slight_smile: Otherwise, I’ll take a look.

2 Me gusta

I’ll be very busy until next week, so if this isn’t very urgent I can take a look then.

2 Me gusta

Np I’ll have a look into this.

3 Me gusta

Submitted a PR first as I needed to introduce a new extension to our PostRevisor class.

https://github.com/discourse/discourse/pull/4380

4 Me gusta