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

Custom modules #2785

Open
wants to merge 48 commits into
base: master
Choose a base branch
from

Lint fix and add '-custom-module' suffix to the id to ensure it is un…

a32aace
Select commit
Loading
Failed to load commit list.
Open

Custom modules #2785

Lint fix and add '-custom-module' suffix to the id to ensure it is un…
a32aace
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Aug 13, 2024 in 1s

45.34% (-0.61%) compared to eb521e6

View this Pull Request on Codecov

45.34% (-0.61%) compared to eb521e6

Details

Codecov Report

Attention: Patch coverage is 8.04598% with 80 lines in your changes missing coverage. Please review.

Project coverage is 45.34%. Comparing base (eb521e6) to head (a32aace).
Report is 639 commits behind head on master.

Files Patch % Lines
website/src/views/timetable/CustomModulesForm.tsx 4.25% 45 Missing ⚠️
website/src/actions/timetables.ts 4.34% 22 Missing ⚠️
...ite/src/views/timetable/CustomModulesContainer.tsx 0.00% 10 Missing ⚠️
website/src/reducers/moduleBank.ts 0.00% 2 Missing ⚠️
website/src/views/timetable/TimetableContent.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2785      +/-   ##
==========================================
- Coverage   45.95%   45.34%   -0.61%     
==========================================
  Files         248      250       +2     
  Lines        5273     5359      +86     
  Branches     1222     1234      +12     
==========================================
+ Hits         2423     2430       +7     
- Misses       2850     2929      +79     

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