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

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

hl - 6pm-4 Add ability to upload CSV of courses to courses page #235

Closed soyflourbread closed 3 years ago

soyflourbread commented 3 years ago

Add an upload button for uploading courses CSV.

Actually, both the functionalities and most tests are already in the codebase, and they all work. I have no idea what to do.

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 73da1ae44725e389b7853f6063a209cd4eb69848

codecov[bot] commented 3 years ago

Codecov Report

Merging #235 (73da1ae) into main (76a482a) will decrease coverage by 0.00%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #235      +/-   ##
============================================
- Coverage     99.82%   99.81%   -0.01%     
  Complexity      342      342              
============================================
  Files            83       83              
  Lines          1669     1659      -10     
  Branches         77       77              
============================================
- Hits           1666     1656      -10     
  Misses            3        3              
Impacted Files Coverage Δ Complexity Δ
javascript/src/main/components/Admin/LoginTable.js 100.00% <0.00%> (ø) 0.00% <0.00%> (ø%)
...ascript/src/main/components/Courses/CourseTable.js 100.00% <0.00%> (ø) 0.00% <0.00%> (ø%)
...src/main/components/OfficeHours/OfficeHourTable.js 100.00% <0.00%> (ø) 0.00% <0.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 76a482a...73da1ae. Read the comment docs.

WadeVaresio commented 3 years ago

My apologies, this issue has already been implemented, please see your teams Kanban board for a new story.