Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: switchScrollingEffect call error when getContainer is false #122

Merged
merged 1 commit into from Dec 13, 2019

Conversation

shaodahong
Copy link
Member

@vercel
Copy link

vercel bot commented Dec 13, 2019

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/react-component/rc-drawer/e3qxsyihc
🌍 Preview: https://rc-drawer-git-fix-get-container.react-component.now.sh

@codecov
Copy link

codecov bot commented Dec 13, 2019

Codecov Report

Merging #122 into master will increase coverage by 0.7%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #122     +/-   ##
========================================
+ Coverage    66.4%   67.1%   +0.7%     
========================================
  Files           3       3             
  Lines         375     377      +2     
  Branches      146     146             
========================================
+ Hits          249     253      +4     
+ Misses        126     124      -2
Impacted Files Coverage Δ
src/DrawerChild.tsx 62.58% <100%> (+0.99%) ⬆️

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 f186eef...3ce7d67. Read the comment docs.

@afc163 afc163 merged commit 9e3a01c into master Dec 13, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix-get-container branch December 13, 2019 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants