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] OWRank: Fix autocommit #2685

Merged
merged 3 commits into from
Oct 17, 2017
Merged

[FIX] OWRank: Fix autocommit #2685

merged 3 commits into from
Oct 17, 2017

Conversation

lanzagar
Copy link
Contributor

Issue

Selecting rows was sending new selections even when auto-send was unchecked.

@lanzagar lanzagar assigned lanzagar and mstrazar and unassigned lanzagar Oct 16, 2017
@codecov-io
Copy link

codecov-io commented Oct 16, 2017

Codecov Report

Merging #2685 into master will increase coverage by <.01%.
The diff coverage is 96%.

@@            Coverage Diff             @@
##           master    #2685      +/-   ##
==========================================
+ Coverage   75.82%   75.82%   +<.01%     
==========================================
  Files         338      338              
  Lines       59459    59478      +19     
==========================================
+ Hits        45082    45101      +19     
  Misses      14377    14377

@janezd
Copy link
Contributor

janezd commented Oct 16, 2017

Could you add some tests?

Actually, I added them. (You're welcome. ;) )

@lanzagar
Copy link
Contributor Author

(Thank you :))

@janezd janezd merged commit ab97e2c into biolab:master Oct 17, 2017
@lanzagar lanzagar deleted the owrank branch March 14, 2022 14:41
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.

4 participants