This has been fixed in the latest deploy. What was going on was pretty subtle.
Basically:
- Before applying a change we always re-validate it
- When a change is applied, all other outstanding changes that affect the same documents are merged with the now-applied change
- If an attempt to merge a change fails, the outstanding change is rejected as conflicting* with an approved change
The bug was it was possible for a merge to succeed, but result in a "change" that didn't actually modify anything. When the change was then reviewed, validation would be run and it would fail.
We now correctly detect this case, and reject changes as conflicting appropriately. Old proposed changes that got into this state have been cleaned up as well.
*Rejected as conflicting is different from normal rejection, as it doesn't indicate that there was anything wrong with the change - it just cannot be applied. The Proposed Change Review Queue does not yet display this state as distinct from normal rejection, but will soon.