Closed natalynyu closed 1 year ago
Merging #649 (3fe58f1) into develop (ae69a21) will decrease coverage by
0.99%
. The diff coverage is79.06%
.:exclamation: Current head 3fe58f1 differs from pull request most recent head 213844c. Consider uploading reports for the commit 213844c to get more accurate results
@@ Coverage Diff @@
## develop #649 +/- ##
===========================================
- Coverage 95.07% 94.09% -0.99%
===========================================
Files 194 194
Lines 4511 4587 +76
Branches 545 561 +16
===========================================
+ Hits 4289 4316 +27
- Misses 104 150 +46
- Partials 118 121 +3
Impacted Files | Coverage Δ | |
---|---|---|
src/server/faculty/faculty.module.ts | 100.00% <ø> (ø) |
|
...t/components/pages/Courses/ReportDownloadModal.tsx | 74.00% <50.00%> (-18.16%) |
:arrow_down: |
src/client/api/reports.ts | 12.90% <60.00%> (-79.96%) |
:arrow_down: |
src/server/report/report.controller.ts | 74.19% <60.00%> (-6.76%) |
:arrow_down: |
...rc/client/components/pages/Faculty/FacultyPage.tsx | 90.07% <70.00%> (-3.73%) |
:arrow_down: |
src/server/report/report.service.ts | 93.24% <88.88%> (-6.76%) |
:arrow_down: |
...rc/client/components/pages/Courses/CoursesPage.tsx | 92.03% <100.00%> (ø) |
|
.../components/pages/Schedule/blocks/SessionBlock.tsx | 91.42% <100.00%> (ø) |
|
src/server/report/report.module.ts | 100.00% <100.00%> (ø) |
|
... and 2 more |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR adds in a Download Faculty Report button, which allows the user to specify a range of academic years in order to generate a faculty report for those years. The data in the report mirrors the table displayed in the Faculty Schedule table.
Types of changes
Checklist:
eslint
on the codePriority:
Related Issues:
Fixes #641