Review on package dependencies
cintnguyen opened this issue · 4 comments
cintnguyen commented
When referencing requirements for an MFE here , 13 is to only have hard dependencies on libraries that are either in the open edX organization or a commonly used third-party (generic NPM package) . During discovery found that:
- "@optimizely/react-sdk" requires API keys making it not readily accessible to the community, referenced here. Is this something we can remove and replace? On npm, it states that it is an "A/B testing and feature management tool for product development teams that enables you to experiment at every step"
- "react-zendesk" issues outlined in #225 , wanted to bring attention to it. Not sure if we have something internal to replace
- "@wojtekmaj/enzyme-adapter-react-17" is the unofficial adapter for react 17 from enzyme. Currently an inactive project. Can this be removed?
justinhynes commented
Thanks for bring attention to this. I have opened up an internal 2U ticket for Aperture to take a look at this and determine a path forward. It will be tracked as ticket APER-3568.
MaxFrank13 commented
Optimizely removal has been merged: #386
MaxFrank13 commented
Enzyme adapter removal: #276
MaxFrank13 commented
Zendesk replacement/removal will be handled by the private ticket linked in #225