Altinn / altinn-studio

Next generation open source Altinn platform and applications.
https://docs.altinn.studio
BSD 3-Clause "New" or "Revised" License
109 stars 73 forks source link

Update bpmn-js ^v17 #12975

Open standeren opened 2 weeks ago

standeren commented 2 weeks ago

Description

Bpmn-js has released a new major version (we are upgrading from v13 -> v17) and currently some of our unit tests are failing due to usage of the BpmnModeler from bpmn-js that apparently has a failing import of a depending package path-intersection. Seems like this is due to an error inside the bpmn-js package which we cannot implement a work-around for so we are temporarily postponing this update until next release.

Here is the renovate PR

Things that have been tested to analyze the issue:

  1. Investigating the dependency package --> no obvious source for our failing tests (or the failing import)
  2. Checking for any similar bug-reports --> could not find any
  3. Declaring the depending packages as dependencies in our package.json file as well --> did not help
  4. Updating one major version at a time to see where the error occurred. Both 14 and 15 had a similar error but with the diagram-js-ui import, while both 16 and 17 had the same path-intersection import error

Additional Information

No response

Tasks

No response

Acceptance Criterias

No response