-
Notifications
You must be signed in to change notification settings - Fork 1
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
Resolve npm dependency issues in edx-platform #26
Comments
[email protected] is the latest version that is using [email protected] so an update is not possible here. I believe it isn't breaking anything at the moment and we can eventually exclude these packages from the project when we will break it down into more MFEs till then we can keep it. |
@abdullahwaheed, @ishahroz: is this related to the Karma vs Jest conversation we had today? Is there another issue that tracks that? |
yes this issue is covered in #105 as well |
Some vague questions:
|
The proctoring issue has been resolved here: openedx/edx-platform#34874 |
|
The edx-platform has conflict in package dependency. This can help to resolve since various process or scripts starts by running
npm install
in the root ofedx-platform
which might take time even ifnode_modules
have been already installed. sincenpm install
runs preinstall cycle anyway REFThis issue might not too related to FEWG. however, it might be helpful to post here since it's related to FEWG type of expertise.
1-
edx/edx-proctoring
MFE-Platform is
edx/[email protected]
whileedx-proctoring-proctortrack
is usingedx/[email protected]
[REF|https://github.com/anupdhabarde/edx-proctoring-proctortrack/blob/d717f3b7e1d5aec6290d7dd08646cba8c6af3499/package.json#L26].Note: I am not sure about how
edx-proctoring-proctortrack
is intended to be installed, the link source returns the master branch here2-
selenium-webdriver
There multiple pacakges in dev depedency that uses the
selenium-webdriver
, however they conflict with each other, for example,karma-selenium-webdriver-launcher
is using[email protected]
REF however the platform it self is using[email protected]
REFOther Relevant Links
The text was updated successfully, but these errors were encountered: