Skip to content
This repository has been archived by the owner on Oct 6, 2020. It is now read-only.

fix(Button): Rename internal loading prop to isLoading to remove console error #102

Merged
merged 2 commits into from
Feb 5, 2020

Conversation

kylealwyn
Copy link
Contributor

@kylealwyn kylealwyn commented Feb 5, 2020

Tired of seeing this console error
Screen Shot 2020-02-04 at 4 33 46 PM

Copy link
Contributor

@alextranwork alextranwork left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor comments

src/Button/Button.js Show resolved Hide resolved
src/Button/Button.js Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Feb 5, 2020

Codecov Report

Merging #102 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #102   +/-   ##
=======================================
  Coverage   66.15%   66.15%           
=======================================
  Files          21       21           
  Lines         458      458           
  Branches       93       93           
=======================================
  Hits          303      303           
  Misses        124      124           
  Partials       31       31

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 3954f8e...d291e49. Read the comment docs.

@kylealwyn kylealwyn merged commit 303879e into master Feb 5, 2020
@kylealwyn kylealwyn deleted the fix/button-loading-prop branch February 5, 2020 00:45
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.

3 participants