openedx/frontend-app-learner-dashboard

Review on package dependencies

cintnguyen opened this issue · 4 comments

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?

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.

Optimizely removal has been merged: #386

Enzyme adapter removal: #276

Zendesk replacement/removal will be handled by the private ticket linked in #225