Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paragon installed as dependency instead of peer dependency #393

Closed
adamstankiewicz opened this issue Jan 11, 2024 · 0 comments · Fixed by #394
Closed

Paragon installed as dependency instead of peer dependency #393

adamstankiewicz opened this issue Jan 11, 2024 · 0 comments · Fixed by #394

Comments

@adamstankiewicz
Copy link
Member

@openedx/paragon is installed as a dependency within package.json for this repo. As a result, Paragon is included as part of the code bundle generated by frontend-component-footer. Given this, consumers (MFEs) using frontend-component-footer are effectively requiring users to download duplicate copies of Paragon (more than likely at different versions, too): one copy of Paragon for the MFE and another copy of Paragon for frontend-component-footer.

That said, because Paragon supports tree shaking, only the components directly used by frontend-component-footer are included in the bundle (i.e., not the entire library).

Generally, Paragon should be treated as a peer dependency in Open edX JS libraries, relying on the installed version of Paragon provided by the consumer (MFE). For some reason, the peer dependency was (incorrectly) removed in favor of a regular dependency in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
1 participant