-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
should solve #2863 #2864
should solve #2863 #2864
Conversation
Thanks for contribution @AndrewKaninchen , you can test it on the preview site. 😊 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov ReportBase: 42.78% // Head: 42.84% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #2864 +/- ##
==========================================
+ Coverage 42.78% 42.84% +0.06%
==========================================
Files 545 545
Lines 25775 25777 +2
Branches 261 261
==========================================
+ Hits 11027 11044 +17
+ Misses 14708 14693 -15
Partials 40 40
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
should solve #2863
I couldn't test it, but it should work if the underlying code works.
If it does, the example code on the website can be changed to implement dropping, which should be pretty straightforward.