Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

feat: allow reactify callbacks to access props #200

Merged
merged 1 commit into from
Aug 5, 2019

Conversation

etr2460
Copy link
Contributor

@etr2460 etr2460 commented Aug 2, 2019

This is needed so that the callbacks passed into this component can perform actions based on props/state on the react component

@xtinec @kristw @williaster

@etr2460 etr2460 requested a review from a team as a code owner August 2, 2019 22:15
@codecov
Copy link

codecov bot commented Aug 2, 2019

Codecov Report

Merging #200 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #200   +/-   ##
=======================================
  Coverage   99.91%   99.91%           
=======================================
  Files          94       94           
  Lines        1165     1165           
  Branches      281      281           
=======================================
  Hits         1164     1164           
  Partials        1        1
Impacted Files Coverage Δ
...ages/superset-ui-chart/src/components/reactify.tsx 100% <100%> (ø) ⬆️

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 32c8ffd...491ec3e. Read the comment docs.

@netlify
Copy link

netlify bot commented Aug 2, 2019

Deploy preview for superset-ui ready!

Built with commit 491ec3e

https://deploy-preview-200--superset-ui.netlify.com

@etr2460 etr2460 merged commit 9d6f5c0 into master Aug 5, 2019
@delete-merged-branch delete-merged-branch bot deleted the erik-ritter--reactify-bind branch August 5, 2019 15:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants