Diffs sequential loading codebase cleanup phase
After the work of #35754 (closed) (and &1816) comes together and everything works as expected, there will be a few things to be cleared when using only the new diffs_batch
endpoint:
-
Projects::MergeRequests::DiffsController
will carry quite some dead code that was being used to render all diffs, so ideally we should remove all@
variables that aren't needed, plus a few private methods. -
Remove the feature flag. -
Double-check if the DiffsSerializer
will be needed (unlikely), then remove it.
Edited by Michelle Gill