ucsb-cs156-s21 / proj-ucsb-cs-las

https://proj-ucsb-cs-las.herokuapp.com/
MIT License
2 stars 3 forks source link

6pm 1 delete tutor assignment #286

Closed omarbuzdar closed 3 years ago

omarbuzdar commented 3 years ago

Added fully functioning delete button for tutor assignments and tests pass.

codecov[bot] commented 3 years ago

Codecov Report

Merging #286 (c772e76) into main (492c8fa) will increase coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #286   +/-   ##
=========================================
  Coverage     99.84%   99.84%           
- Complexity      405      408    +3     
=========================================
  Files            90       90           
  Lines          1928     1944   +16     
  Branches         93       94    +1     
=========================================
+ Hits           1925     1941   +16     
  Misses            3        3           
Impacted Files Coverage Δ
...components/TutorAssignment/TutorAssignmentTable.js 100.00% <100.00%> (ø)
...src/main/pages/TutorAssignment/TutorAssignments.js 100.00% <100.00%> (ø)
...services/TutorAssignment/TutorAssignmentService.js 100.00% <100.00%> (ø)
...sbcslas/controllers/TutorAssignmentController.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 492c8fa...c772e76. Read the comment docs.

github-actions[bot] commented 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.

PR review process

Generated by :no_entry_sign: dangerJS against c772e7662eea4f4ae8cbdcda349006feb802e1da

kaihilbourne commented 3 years ago

I figured out how to use github's conflict resolver thing, and chose the correct way to merge this branch with main.