Question: Is there any reason why token<-> ICO relation is not implemented through ModuleHandler?

CORIONplatform / solidity

GNU General Public License v3.0

12 stars 9 forks source link

Question: Is there any reason why token<-> ICO relation is not implemented through ModuleHandler? #117

Closed gundas closed 7 years ago

gundas commented 7 years ago

It seems that the idea of ModuleHandler was to take care of a communication among the modules? So, for example the Token could call ModuleHandler to figure out if ICO is ongoing or not. Also if ICO was a normal module, it could also process Transfer events to figure out changes in InterestDB etc. - that would reduce the coupling among the modules.

On the other hand - I guess the change might be too big at the current stage.

iFA88 commented 7 years ago

ICO can directly communicate with token and premium I think this is ok now. :)