Unify hashDigest #75

New Issue

2019-04-05T16:41:14Z

fsmanuel commented

2019-04-05 16:41:14 +00:00

(Migrated from github.com)

Can we address this issue as well:

https://github.com/67P/kredits-contracts/blob/93e0b30b357e69a892b0c3187a813b95f0a1d619/lib/contracts/contributor.js#L24-L25

I have 17 9 ipfsHash matches in https://github.com/67P/kredits-contracts/blob/master/apps/contributor/contracts/Contributor.sol

Update:
I think we don't use this function at all:
https://github.com/67P/kredits-contracts/blob/93e0b30b357e69a892b0c3187a813b95f0a1d619/apps/contributor/contracts/Contributor.sol#L72-L80

Can we address this issue as well: https://github.com/67P/kredits-contracts/blob/93e0b30b357e69a892b0c3187a813b95f0a1d619/lib/contracts/contributor.js#L24-L25 I have ~17~ 9 `ipfsHash` matches in https://github.com/67P/kredits-contracts/blob/master/apps/contributor/contracts/Contributor.sol Update: I think we don't use this function at all: https://github.com/67P/kredits-contracts/blob/93e0b30b357e69a892b0c3187a813b95f0a1d619/apps/contributor/contracts/Contributor.sol#L72-L80

👍 1

raucao commented

2019-04-05 18:25:26 +00:00

(Migrated from github.com)

PRs welcome!

bumi commented

2019-04-05 23:16:21 +00:00

(Migrated from github.com)

actually I have this code that I just commited: https://github.com/67P/kredits-contracts/pull/79
but this needs testing...

actually I have this code that I just commited: https://github.com/67P/kredits-contracts/pull/79 but this needs testing...

👍 1

Sign in to join this conversation.