Skip to content

fix: import api to chunk file - #734

Merged
KristinAoki merged 1 commit into
masterfrom
KristinAoki/fix-413-error-on-import-upload
Dec 12, 2023
Merged

fix: import api to chunk file#734
KristinAoki merged 1 commit into
masterfrom
KristinAoki/fix-413-error-on-import-upload

Conversation

@KristinAoki

@KristinAoki KristinAoki commented Dec 7, 2023

Copy link
Copy Markdown
Contributor

JIRA Ticket: TNL-11163

I was able to import a 60MB course. When I tried to do a 250MB course it broke. I tried a bit smaller and it seemed to break around 100MB. It looked like the percent was going up. The percent got to 99%, then the percentage disappeared and the gear would just spin, see screenshots. The legacy page they can import a 500 MB course but with the new page cannot import 100 MB.

** This PR is dependent on edx-platform PR #33898 **

Testing

  1. Navigate to the Import page
  2. If you wan to keep the content of your current course, export it before importing a new one
  3. Upload a course zip that is greater than 100 MB
  4. File should import
  5. Wait for the import to finish
  6. Click "View updated outline"
  7. Check that the course properly imported

@codecov

codecov Bot commented Dec 7, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (04c1427) 88.85% compared to head (7aa2e29) 88.83%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #734      +/-   ##
==========================================
- Coverage   88.85%   88.83%   -0.02%     
==========================================
  Files         468      468              
  Lines        7168     7185      +17     
  Branches     1539     1541       +2     
==========================================
+ Hits         6369     6383      +14     
- Misses        772      775       +3     
  Partials       27       27              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

rayzhou-bit
rayzhou-bit previously approved these changes Dec 8, 2023
@rayzhou-bit
rayzhou-bit dismissed their stale review December 8, 2023 20:11

need code cov

@rayzhou-bit
rayzhou-bit self-requested a review December 8, 2023 20:11

@rayzhou-bit rayzhou-bit left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good but needs codecov

@rayzhou-bit
rayzhou-bit self-requested a review December 8, 2023 20:14
@KristinAoki
KristinAoki merged commit 6f7a992 into master Dec 12, 2023
@KristinAoki
KristinAoki deleted the KristinAoki/fix-413-error-on-import-upload branch December 12, 2023 15:28
jesperhodge added a commit that referenced this pull request Dec 12, 2023
jesperhodge added a commit that referenced this pull request Dec 12, 2023
Internal issue: https://2u-internal.atlassian.net/servicedesk/customer/portal/9/CR-6328?created=true

Reverted 6 merged PRs due to problems.

scroll was not working on editors
potential problems with editor content loading

------------------------------------------------------


* Revert "fix(deps): update dependency @edx/frontend-lib-content-components to v1.177.4 (#742)"

This reverts commit cc40e9d.

* Revert "feat: add escalation email field for LTI-based proctoring providers (#736)"

This reverts commit 0f483dc.

* Revert "fix: video downloads (#728)"

This reverts commit c5abd21.

* Revert "fix: import api to chunk file (#734)"

This reverts commit 6f7a992.

* Revert "feat: Taxonomy delete dialog (#684)"

This reverts commit 1eff489.

* Revert "fix(deps): update dependency @edx/frontend-lib-content-components to v1.177.1 (#727)"

This reverts commit dcabb77.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants