search

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

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

6pm-4-Fixed link issue on office hours where the link would go outside of the table (Remade PR) #309

Closed bzamora020 closed 3 years ago

bzamora020 commented 3 years ago

In this PR, I fixed the link issue on office hours where the link would go outside of the table. This PR had already been done before but due to merging conflicts, I dropped the branch and created a new branch from the fresh main.

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 c3787eda583a7b19f7afff34f4b9b9ed69aff268

codecov[bot] commented 3 years ago

Codecov Report

Merging #309 (c3787ed) into main (de16512) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #309   +/-   ##
=========================================
  Coverage     99.75%   99.75%           
  Complexity      406      406           
=========================================
  Files            99       99           
  Lines          2078     2078           
  Branches         97       97           
=========================================
  Hits           2073     2073           
  Misses            5        5
Impacted Files Coverage Δ
...src/main/components/OfficeHours/OfficeHourTable.js 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 de16512...c3787ed. Read the comment docs.