Skip to content

WIP: Just clear the markdown cache when saving objects

Nick Thomas requested to merge (removed):remove-md-cache-update-on-save into master

What does this MR do?

Changes the behaviour of object.save so markdown cache columns are cleared, rather than updated

Are there points in the code the reviewer needs to double check?

We're moving work from the update phase to the first view phase. Since we generally redirect the updater back to view their own content, this will be a no-op most of the time, although I'm unsure about highly concurrent viewing scenarios.

Why was this MR needed?

In https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1708 we discovered that the existing behaviour breaks some specs in a hard-to-understand way.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

Merge request reports