Merged edits will clobber the first "edit reason"

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.

4 Likes

@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.

3 Likes

A fix for this has now been merged! :hammer:

4 Likes