Skip to content

Remove Balsamiq File Preview

Lukas Eipert requested to merge leipert-remove-balsamiq into master

What does this MR do and why?

The Balsamiq File Preview is not utilized a lot. In fact it has been broken since over a year on gitlab.com and on self managed instances #285525 (closed). The Balsamiq preview itself doesn't have a lot of value as it just renders the preview of the first Balsamiq page and not all the pages. We have received no user complaints whatsoever, meaning that likely noone relied on this feature.

This MR removes the previewer, as it is also riddled with tech debt, for example shipping a asm.js compiled version of sql.js

Changelog: removed

Closes #300869 (closed)

Closes #324334 (closed)

Closes #285525 (closed)

Screenshots or screen recordings

If the feature were working Currently in Omnibus / gitlab.com After the removal
Screenshot_2022-03-15_at_18.01.59 Screenshot_2022-03-15_at_17.59.27 Screenshot_2022-03-15_at_17.59.09

How to set up and validate locally

N/A

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Lukas Eipert

Merge request reports

Loading