# Merged edits will clobber the first "edit reason"

**URL:** https://meta.discourse.org/t/merged-edits-will-clobber-the-first-edit-reason/123475
**Category:** Bug
**Created:** [July 19, 2019, 10:11pm UTC](https://meta.discourse.org/t/merged-edits-will-clobber-the-first-edit-reason/123475 "2019-07-19T22:11:24Z")
**Posts on this page:** 4
**Page:** 1

<div class="post-metadata">

### Author: ![supermathie](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/supermathie/32/507518_2.png) [@supermathie](https://meta.discourse.org/u/supermathie)
#### Post date: [July 19, 2019, 10:11pm UTC](https://meta.discourse.org/t/merged-edits-will-clobber-the-first-edit-reason/123475/1 "2019-07-19T22:11:24Z")

</div>

If a user edits a post and includes an Edit Reason then (within the edit window) makes another edit without entering a reason, the first Edit Reason will be clobbered.

Edit Reasons should probably concatenate instead.

---

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@sam](https://meta.discourse.org/u/sam)
#### Post date: [November 12, 2019, 1:24am UTC](https://meta.discourse.org/t/merged-edits-will-clobber-the-first-edit-reason/123475/2 "2019-11-12T01:24:19Z")

</div>

@Martin_Brennan seems like an easy fix.

I would not go so far as to concatenate here, instead if a reason is specified then a revision needs to be forced somehow.

- If this is the first edit… you got to add a revision so you can keep the text of the edit reason somewhere.

- If it is a subsequent edit… you can still handle grace period properly as long as you don’t change the reason.

- Once you change the reason you force a new edit.

---

<div class="post-metadata">

### Author: ![martin](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/martin/32/491371_2.png) [@martin](https://meta.discourse.org/u/martin)
#### Post date: [November 18, 2019, 3:18am UTC](https://meta.discourse.org/t/merged-edits-will-clobber-the-first-edit-reason/123475/5 "2019-11-18T03:18:35Z")

</div>

A fix for this has now been merged! 🔨

---

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@sam](https://meta.discourse.org/u/sam)
#### Post date: [November 18, 2019, 3:35am UTC](https://meta.discourse.org/t/merged-edits-will-clobber-the-first-edit-reason/123475/6 "2019-11-18T03:35:22Z")

</div>


