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

Remove soop #417

Merged
merged 7 commits into from
Jul 10, 2014
Merged

Remove soop #417

merged 7 commits into from
Jul 10, 2014

Conversation

maraoz
Copy link
Contributor

@maraoz maraoz commented Jul 10, 2014

... and add some small tests (added while making everything work again without soop)

fixes https://github.com/bitpay/copay/issues/894
fixes #413

@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) when pulling 98d2445 on maraoz:remove/soop2 into 3f0dd8d on bitpay:master.

@ryanxcharles
Copy link
Contributor

Great. Reviewing.

} catch (e) {
return false;
}
};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha ... this is a duplicate of the above function. Good catch.

@ryanxcharles ryanxcharles merged commit 98d2445 into bitpay:master Jul 10, 2014
LinusU added a commit to LinusU/bitcore that referenced this pull request Jul 12, 2014
Removed some unnecessary parenthesise that hung around after the merge
of bitpay#417
@maraoz maraoz deleted the remove/soop2 branch March 13, 2015 15:24
micahriggan pushed a commit to micahriggan/bitcore that referenced this pull request Dec 27, 2018
micahriggan pushed a commit to micahriggan/bitcore that referenced this pull request Dec 27, 2018
Removed some unnecessary parenthesise that hung around after the merge
of bitpay#417
micahriggan pushed a commit to micahriggan/bitcore that referenced this pull request Dec 27, 2018
fix copayerName decryption in certain cases
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.

remove soop from bitcore Consider removing soop
3 participants