Closed kenny-wang6 closed 3 years ago
Nice PR! Your PR can be merged once you get an approving review from a staff member and a member of your team.
Keep in mind, code reviews only count if you use the "Approve" or "Request changes" option in GitHub. You can find this under the "Files changed" tab under the "Review changes" button.
Generated by :no_entry_sign: dangerJS against edf2d7a20126490ec1500cfacd028ce28b2ca2de
Merging #284 (edf2d7a) into main (9f0f0ec) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #284 +/- ##
=========================================
Coverage 99.83% 99.83%
Complexity 390 390
=========================================
Files 87 87
Lines 1820 1822 +2
Branches 85 85
=========================================
+ Hits 1817 1819 +2
Misses 3 3
Impacted Files | Coverage Δ | |
---|---|---|
...src/main/components/OfficeHours/OfficeHourTable.js | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 9f0f0ec...edf2d7a. Read the comment docs.
Added tutor name, course name, and quarter to office hour table. Previously only showed tutor assignment id, which made it difficult to tell who was hosting the office hour and which course it was for.