bpmn-io / dmn-migrate

Migrate your DMN diagrams to the latest DMN version
https://www.npmjs.com/package/@bpmn-io/dmn-migrate
MIT License
4 stars 2 forks source link

Ensure that we properly handle all sorts of broken inputs #16

Closed nikku closed 4 years ago

nikku commented 4 years ago

TLDR: When using the utility users must not accidentally swallow errors or warnings.

What should we do?

Our dmn-js toolkit does, out of the box, handle all sorts of broken input with appropriate error messages during import:

Expected Behavior

Why should we do it?

We advertise the migration utility as a pre-processing step before using dmn-js@8.0.0.

diagramXML -> dmn-migrate -> dmn-js#importXML

In order for that to properly work we must handle errors and warnings appropriately.

philippfromme commented 4 years ago

👍