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

**URL:** https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370
**Category:** Bug
**Created:** [August 8, 2016, 7:17pm UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370 "2016-08-08T19:17:40Z")
**Posts on this page:** 10
**Page:** 1

<div class="post-metadata">

### Author: ![meglio](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/meglio/32/71444_2.png) [@meglio](https://meta.discourse.org/u/meglio)
#### Post date: [August 8, 2016, 7:17pm UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/1 "2016-08-08T19:17:40Z")

</div>

After merging the two posts as on the following screenshot:

 ![](https://global.discourse-cdn.com/meta/original/3X/b/1/b10201006755f20d5b68c54843b3472fbb8032c4.jpg)

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.

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [August 8, 2016, 11:13pm UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/2 "2016-08-08T23:13:27Z")

</div>

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

---

<div class="post-metadata">

### Author: ![meglio](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/meglio/32/71444_2.png) [@meglio](https://meta.discourse.org/u/meglio)
#### Post date: [August 9, 2016, 7:01am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/3 "2016-08-09T07:01:42Z")

</div>

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

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [August 9, 2016, 7:29am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/4 "2016-08-09T07:29:08Z")

</div>

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

---

<div class="post-metadata">

### Author: ![meglio](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/meglio/32/71444_2.png) [@meglio](https://meta.discourse.org/u/meglio)
#### Post date: [August 9, 2016, 7:36am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/5 "2016-08-09T07:36:10Z")

</div>

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.

---

<div class="post-metadata">

### Author: ![tgxworld](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tgxworld/32/106117_2.png) [@tgxworld](https://meta.discourse.org/u/tgxworld)
#### Post date: [August 9, 2016, 7:53am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/6 "2016-08-09T07:53:08Z")

</div>

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

---

<div class="post-metadata">

### Author: ![adrapereira](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/adrapereira/32/51755_2.png) [@adrapereira](https://meta.discourse.org/u/adrapereira)
#### Post date: [August 9, 2016, 9:03am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/7 "2016-08-09T09:03:30Z")

</div>

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

---

<div class="post-metadata">

### Author: ![tgxworld](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tgxworld/32/106117_2.png) [@tgxworld](https://meta.discourse.org/u/tgxworld)
#### Post date: [August 9, 2016, 10:56am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/8 "2016-08-09T10:56:53Z")

</div>

Np I’ll have a look into this.

---

<div class="post-metadata">

### Author: ![tgxworld](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tgxworld/32/106117_2.png) [@tgxworld](https://meta.discourse.org/u/tgxworld)
#### Post date: [August 10, 2016, 6:05am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/9 "2016-08-10T06:05:54Z")

</div>

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

[https://github.com/discourse/discourse/pull/4380](https://github.com/discourse/discourse/pull/4380)

---

<div class="post-metadata">

### Author: ![tgxworld](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tgxworld/32/106117_2.png) [@tgxworld](https://meta.discourse.org/u/tgxworld)
#### Post date: [August 13, 2016, 9:05am UTC](https://meta.discourse.org/t/merging-two-posts-with-a-new-merge-feature-does-not-update-posts-in-place-as-expected/48370/10 "2016-08-13T09:05:30Z")

</div>


