TC_AUTHOR_88 Upload files to course #166
Assignees
Milestone
Comments
|
A PR that might solve is open here openedx/edx-platform#30309 |
ghassanmas
added a commit
to ghassanmas/studio-frontend
that referenced
this issue
Apr 28, 2022
The PR openedx#315 added @babel/core >= 7.*. upgrading/using babel v7 requires, making certian changes, e.g using @babel...etc. This lead to `npm run build` being interrupted/exited.without generating the /dist folder. Luckily, there is a pacakge to automate migration of config using `npx babel-upgrade --write`, followed by `npm install`, and few minor changes, the dist folder came to life again. This shall fixes this openedx/wg-build-test-release#166. and probably making this openedx/edx-platform#30309 unnecessary.
|
Also possible root fix is proposed at openedx/studio-frontend#341 |
ghassanmas
added a commit
to ghassanmas/studio-frontend
that referenced
this issue
May 12, 2022
The PR openedx#315 added @babel/core >= 7.*. upgrading/using babel v7 requires, making certian changes, e.g using @babel...etc. This lead to `npm run build` being interrupted/exited.without generating the /dist folder. Luckily, there is a pacakge to automate migration of config using `npx babel-upgrade --write`, followed by `npm install`, and few minor changes, the dist folder came to life again. This shall fixes this openedx/wg-build-test-release#166. and probably making this openedx/edx-platform#30309 unnecessary. Note: This change adds "^7.0.0-bridge.0" as devDep, this is needed because we have pacakges that uses Babel 6, which are jest and babel-jest. once we upgrade to babel-jest and jest >=24.*.* the pacakge shall be removed.
aht007
pushed a commit
to openedx/studio-frontend
that referenced
this issue
May 13, 2022
The PR #315 added @babel/core >= 7.*. upgrading/using babel v7 requires, making certian changes, e.g using @babel...etc. This lead to `npm run build` being interrupted/exited.without generating the /dist folder. Luckily, there is a pacakge to automate migration of config using `npx babel-upgrade --write`, followed by `npm install`, and few minor changes, the dist folder came to life again. This shall fixes this openedx/wg-build-test-release#166. and probably making this openedx/edx-platform#30309 unnecessary. Note: This change adds "^7.0.0-bridge.0" as devDep, this is needed because we have pacakges that uses Babel 6, which are jest and babel-jest. once we upgrade to babel-jest and jest >=24.*.* the pacakge shall be removed.
|
Since this openedx/studio-frontend#341 is merged and also tutor has cherry picked this openedx/edx-platform#30309 . I think its safe to close this |
|
This is now marked as Passed and Resolved in the test sheet. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I wasn't able to test files uploads in studio for any course, this due a bug where certian statics files wan't served i.e. 404 return error.
Please Note: since this test fails it also going to block the testing for TC_AUTHOR_88 to TC_AUTHOR_102. Sice it's even not possible to try to upload any file. it's just a blank page.
This issue is not new was repoted in tutor forums before, please check the resources below.
1- https://discuss.overhang.io/t/missing-js-css-files-missing-from-openedx-docker-image-in-studio/2629/18
2- https://discuss.overhang.io/t/files-uploads-page-is-blank/2631/3
The text was updated successfully, but these errors were encountered: