Skip to content
This repository was archived by the owner on Jan 16, 2022. It is now read-only.

Card Component - Introduced ForwardRef#216

Merged
juanpicado merged 2 commits intomasterfrom
refactor/162_introduced_forward_ref_card
Oct 26, 2019
Merged

Card Component - Introduced ForwardRef#216
juanpicado merged 2 commits intomasterfrom
refactor/162_introduced_forward_ref_card

Conversation

@priscilawebdev
Copy link
Copy Markdown
Contributor

Type: Refactor

The following has been addressed in the PR: #162

Description:

Wrapped Material UI Card Component with ForwardRef.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 25, 2019

Codecov Report

Merging #216 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #216      +/-   ##
==========================================
+ Coverage   87.52%   87.55%   +0.02%     
==========================================
  Files         117      118       +1     
  Lines         874      876       +2     
  Branches      139      160      +21     
==========================================
+ Hits          765      767       +2     
+ Misses         97       94       -3     
- Partials       12       15       +3
Impacted Files Coverage Δ
src/components/Help/styles.ts 100% <ø> (ø) ⬆️
src/components/DetailSidebar/DetailSidebar.tsx 85% <ø> (ø) ⬆️
src/components/Dependencies/styles.ts 100% <ø> (ø) ⬆️
src/muiComponents/Card/Card.tsx 100% <100%> (ø)
src/utils/styles/media.ts 81.81% <0%> (ø) ⬆️

@priscilawebdev priscilawebdev changed the title Card Components - Introduced ForwardRef Card Component - Introduced ForwardRef Oct 25, 2019
@juanpicado juanpicado merged commit 2bc49f3 into master Oct 26, 2019
@delete-merged-branch delete-merged-branch bot deleted the refactor/162_introduced_forward_ref_card branch October 26, 2019 06:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants