Fixes the removal of redundant contribution and update related issues #4086

Merged
merged 2 commits into from Nov 3, 2016

Conversation

Projects
None yet
2 participants
@Akarshit
Member

Akarshit commented Oct 31, 2015

FIx for #4082 , #4084

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Feb 13, 2016

Member

But from the comments, this appears to introduce a new (major?) bug:

// TODO: Every contribution is getting added twice
// and nothing is replaced ever.

or was that there before?

Member

benfry commented Feb 13, 2016

But from the comments, this appears to introduce a new (major?) bug:

// TODO: Every contribution is getting added twice
// and nothing is replaced ever.

or was that there before?

@Akarshit

This comment has been minimized.

Show comment
Hide comment
@Akarshit

Akarshit Apr 22, 2016

Member

No that was already happening before, I just added the comment so that it can be looked into.

Member

Akarshit commented Apr 22, 2016

No that was already happening before, I just added the comment so that it can be looked into.

@benfry benfry merged commit c5bfec6 into processing:master Nov 3, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment