MauroDataMapper/mdm-ui

Remove legacy branch merge UI components

Closed this issue · 0 comments

pjmonks commented

A MergeDiffContainerComponent was created a while ago to redesign the branching/merge diff tooling in mdm-ui. This was kept behind a feature toggle so the original ModelMergingComponent legacy UI could still be used. However, the newer UI has now become the standard by default, so the older UI can be removed.

This will have the benefit of:

  1. Being able to remove the useMergeUiV2 feature toggle
  2. Reduce the compiled size of the Angular code by removing unused code
  3. Remove some code quality issues found via Sonarqube: https://jenkins.cs.ox.ac.uk/sonarqube/project/issues?id=mdm-ui&open=AXlb23vt810i_loXyEKK&resolved=false&severities=MAJOR&types=BUG
  4. Reducing the overall complexity of the repo