dorinori commented 3 years ago

Added download csv button on tutors page that downloads all tutor data onto local machine

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 78545441c14a07225a9e4a30e15e8567c735e4fa

codecov[bot] commented 3 years ago

Codecov Report

Merging #253 (7854544) into main (58cb120) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main     #253   +/-   ##
=========================================
  Coverage     99.82%   99.82%           
  Complexity      357      357           
=========================================
  Files            84       84           
  Lines          1706     1707    +1     
  Branches         78       78           
=========================================
+ Hits           1703     1704    +1     
  Misses            3        3

Impacted Files	Coverage Δ
javascript/src/main/pages/Tutor/Tutor.js	`88.46% <100.00%> (+0.46%)`	:arrow_up:

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 58cb120...7854544. Read the comment docs.

soyflourbread commented 3 years ago

LGTM

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

6pm-4-add download csv button on tutors page #253

Codecov Report