yesamer commented on PR #2203: URL: https://github.com/apache/incubator-kie-tools/pull/2203#issuecomment-2008999422
@tiagobento Thank you for your quick reaction to fix that issue. @gitgabrio The solution of having our custom xml <--> json marshaller was largely discussed and agreed upon with the tooling team. I agree with you that reinventing the wheel should always be avoided, but in this case.. we didn't have a wheel! It can be hard to believe, but there's no reliable, well-maintained xml <--> json marshaller in npm libraries. @pefernan spent a lot of time analyzing and assessing all the available solutions, and no one fits our needs. Hard to believe but that is. I checked that by myself and I confirm that. [jsonx](https://github.com/highsource/jsonix) you mentioned would be a good candidate... but the project is dead, sadly because the [project's maintainer is dead](https://github.com/highsource/jsonix/issues/255). Regarding the quality and reliability of the solution, we tested it with a very representative DMN file, and the results were very promising from the beginning. The new DMN editor is based on that marshaller, and I believe that everyone working on that can confirm the reliability of the marshaller. The new scesim editor is based on that marshaller as well, and I can confirm its reliability. I found just a bug that Tiago immediately fixed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
