Closed m-cabello 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 937d190c08b9ab94c33f5b54865e47577b511701
Merging #266 (1eb57e4) into main (114605a) will decrease coverage by
0.28%
. The diff coverage is94.44%
.:exclamation: Current head 1eb57e4 differs from pull request most recent head 09e322e. Consider uploading reports for the commit 09e322e to get more accurate results
@@ Coverage Diff @@
## main #266 +/- ##
============================================
- Coverage 99.80% 99.51% -0.29%
Complexity 410 410
============================================
Files 99 97 -2
Lines 2058 2066 +8
Branches 94 94
============================================
+ Hits 2054 2056 +2
- Misses 4 10 +6
Impacted Files | Coverage Δ | |
---|---|---|
javascript/src/main/App.js | 100.00% <ø> (ø) |
|
javascript/src/main/components/Nav/AppNavbar.js | 100.00% <ø> (ø) |
|
...t/src/main/components/TutorNotes/TutorNotesForm.js | 83.33% <83.33%> (ø) |
|
.../src/main/components/TutorNotes/TutorNotesTable.js | 100.00% <100.00%> (ø) |
|
...script/src/main/pages/TutorNotes/EditTutorNotes.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%> (ø) |
|
javascript/src/main/utils/CourseFormHelpers.js | 100.00% <0.00%> (ø) |
|
... and 19 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 114605a...09e322e. Read the comment docs.
Duplicate of #308
Implemented the front end components of the tutor notes page