lblod / frontend-embeddable-notule-editor

Frontend for an embeddable variant of the editor.
MIT License
1 stars 1 forks source link

GN-4731: downgrade and pin appuniversum package to 2.15.0 #230

Closed elpoelma closed 8 months ago

elpoelma commented 8 months ago

Overview

Recently (as of version 2.16.0), the @appuniversum/ember-appuniversum started using ember template-imports (https://github.com/ember-template-imports/ember-template-imports) and the .gjs/.gts syntax. While this is definitely the way forward for ember components, it introduces some challenges for this package.

The AuIcon component of the @appuniversum/ember-appuniversum package references svg icons externally. While this is a good approach for applications, it is less useful for packages like these where we want to inline the svgs (to remove the need of hosting the svg icons seperately).

Until recently, we could circumvent this issue by overwriting the appuniversum AuIcon in this package, this solution is no longer always possible with ember-template-imports (you can read https://rfcs.emberjs.com/id/0496-handlebars-strict-mode/ for more information about this).

The temporary solution in this PR is to downgrade the @appuniversum/ember-appuniversum package to version 2.15.0 until we find a more permanent solution.

connected issues and PRs:

GN-4731

How to test/reproduce

Challenges/uncertainties

Downgrading the appuniversum dependency gives us time for a more permanent solution which will probably encompass adding svg-jar support to the appuniversum package.

Checks PR readiness