Closed Zwang1680 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 e0fdb288d5c383135a73ef682ccb78aeee408d62
Merging #308 (e0fdb28) into main (0234335) will increase coverage by
0.05%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #308 +/- ##
============================================
+ Coverage 99.64% 99.69% +0.05%
Complexity 439 439
============================================
Files 102 107 +5
Lines 2229 2304 +75
Branches 104 104
============================================
+ Hits 2221 2297 +76
+ Misses 8 7 -1
Impacted Files | Coverage Δ | |
---|---|---|
javascript/src/main/App.js | 100.00% <ø> (ø) |
|
javascript/src/main/components/Nav/AppNavbar.js | 100.00% <ø> (ø) |
|
...main/components/OfficeHours/OfficeHoursSelector.js | 100.00% <100.00%> (+14.28%) |
:arrow_up: |
...t/src/main/components/TutorNotes/TutorNotesForm.js | 100.00% <100.00%> (ø) |
|
.../src/main/components/TutorNotes/TutorNotesTable.js | 100.00% <100.00%> (ø) |
|
...ascript/src/main/pages/TutorNotes/NewTutorNotes.js | 100.00% <100.00%> (ø) |
|
javascript/src/main/pages/TutorNotes/TutorNotes.js | 100.00% <100.00%> (ø) |
|
.../src/main/services/TutorNotes/TutorNotesService.js | 100.00% <100.00%> (ø) |
|
... and 3 more |
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 0234335...e0fdb28. Read the comment docs.
This is a subset of our previous task to create an entire front end of the tutor notes page instead only creating the front end for the creation of the tutor notes