Closed m-cabello closed 3 years ago
Merging #298 (39df1bd) into main (e6ac6b5) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #298 +/- ##
=========================================
Coverage 99.84% 99.84%
- Complexity 408 410 +2
=========================================
Files 90 90
Lines 1944 1951 +7
Branches 94 94
=========================================
+ Hits 1941 1948 +7
Misses 3 3
Impacted Files | Coverage Δ | |
---|---|---|
...cslas/controllers/OnlineOfficeHoursController.java | 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 e6ac6b5...39df1bd. Read the comment docs.
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 39df1bd8293c04907a89a9f0c07b637667600e3e
Implemented a office hour member route that look up the current user's email, find only the office hours that have an email for that user and return only those. This can be used to populate the office hours in a OfficeHoursSelector component on the TutorNotesForm for entering new office hours.