Review on package dependencies

cintnguyen commented 8 months ago

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 1 month ago

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 1 month ago

Optimizely removal has been merged: https://github.com/openedx/frontend-app-learner-dashboard/pull/386

MaxFrank13 commented 3 weeks ago

Enzyme adapter removal: #276

MaxFrank13 commented 3 weeks ago

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

openedx / frontend-app-learner-dashboard

Review on package dependencies #262