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: 🐛 Fix ArrowAnnotate's event firing on complete/cancel #1266

Merged
merged 1 commit into from
Jul 8, 2020

Conversation

JamesAPetts
Copy link
Member

ArrowAnnotate incorrectly emitted events on cancel, this fixes add being called after a remove.

@codecov
Copy link

codecov bot commented Jul 8, 2020

Codecov Report

Merging #1266 into master will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1266   +/-   ##
=======================================
  Coverage   18.94%   18.94%           
=======================================
  Files         286      286           
  Lines        8647     8647           
  Branches     1476     1476           
=======================================
  Hits         1638     1638           
  Misses       5803     5803           
  Partials     1206     1206           
Impacted Files Coverage Δ
src/tools/annotation/ArrowAnnotateTool.js 0.00% <0.00%> (ø)

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 6693743...c022b38. Read the comment docs.

@JamesAPetts JamesAPetts merged commit 0868812 into master Jul 8, 2020
@dannyrb
Copy link
Member

dannyrb commented Jul 8, 2020

🎉 This PR is included in version 4.18.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants