Skip to content

ASU Moe Teak Release Branch#98

Merged
xitij2000 merged 7 commits intoasu-moe-release/teakfrom
kshitij/asu-moe-release/teak
Jan 21, 2026
Merged

ASU Moe Teak Release Branch#98
xitij2000 merged 7 commits intoasu-moe-release/teakfrom
kshitij/asu-moe-release/teak

Conversation

@xitij2000
Copy link
Copy Markdown
Member

@xitij2000 xitij2000 commented Dec 19, 2025

When we edit & save the component, publish button doesn't show up until we refresh the page manualy or open this unit by opening previous unit and coming back to this unit again.
In this commit, we are dispatching a storage event whenever we edit the component, it'll refresh the page & show the publish button as expected.
@xitij2000 xitij2000 force-pushed the kshitij/asu-moe-release/teak branch from 52d3460 to edb42c2 Compare December 19, 2025 07:01
@xitij2000 xitij2000 marked this pull request as ready for review December 22, 2025 13:42
asadali145 and others added 4 commits December 22, 2025 20:01
) (openedx#2434)

* fix: allow thumbnail upload if no thumbnail

* fix: improve thumbnail upload impl

* test: fix tests

* test: fix tests

* fix: do not show thumbnail upload if not allowed

* test: fix coverage

* test: add thumbnail test

* fix: display thumbnail overlay when video status is success
This handles loading errors when opening the course unit page via direct link as an unauthorized user.

Co-authored-by: Ihor Romaniuk <[email protected]>
commit can be dropped in next release.
@xitij2000 xitij2000 force-pushed the kshitij/asu-moe-release/teak branch from 7909b9a to e1cb877 Compare December 22, 2025 14:32
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 19, 2026

Codecov Report

❌ Patch coverage is 97.43590% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (asu-moe-release/teak@c2e4f7f). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/course-unit/data/thunk.js 93.33% 1 Missing ⚠️
Additional details and impacted files
@@                   Coverage Diff                   @@
##             asu-moe-release/teak      #98   +/-   ##
=======================================================
  Coverage                        ?   93.85%           
=======================================================
  Files                           ?     1152           
  Lines                           ?    24150           
  Branches                        ?     5227           
=======================================================
  Hits                            ?    22667           
  Misses                          ?     1405           
  Partials                        ?       78           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xitij2000 xitij2000 merged commit 7697f27 into asu-moe-release/teak Jan 21, 2026
6 checks passed
@xitij2000 xitij2000 deleted the kshitij/asu-moe-release/teak branch April 16, 2026 11:14
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.

3 participants