jiridanek commented on pull request #852: URL: https://github.com/apache/qpid-dispatch/pull/852#issuecomment-716449695
I've spent some time on this yesterday and the day before, and my conclusion is that the upgrade is doable, and the impact should be localized (as opposed to the PatternFly updates, which seemed harder to me, and required changes to HTML structure of the page in few places). * There are some API renamings/reorganizations which are easy to deal with, * very little removed functionality without replacement (i.e. one predefined color scale) which should not be too hard to replace, * and some significant differences in d3 API that require essentially taking e.g. the new "forces" example and updating the relevant part of the console (the network map) according to the example. The 6.2.0 version of d3 continues moving things around, it for example removed `d3.event`, so I think that trying to catch up to the latest d3 is fools errand. I'll close this dependabot PR and put up a previous version that I played with, which updates to d3 5.16. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
