search

AdChain / AdChainRegistry

adChain registry smart contracts
Apache License 2.0
30 stars 10 forks source link

updateStatus is very hairy #21

Closed skmgoldin closed 6 years ago

skmgoldin commented 7 years ago

Everything it does, it arguably needs to do. But it can definitely be broken out into a few helper functions.

skmgoldin commented 7 years ago

Functions to break out (both internal, no public API changes):

  1. whitelistApplication
  2. resolveChallenge
skmgoldin commented 7 years ago

https://github.com/AdChain/AdChainRegistry/commit/d016f1bb66e9ddd49e65c9b6ec147d2e8057abde