search

mesg-foundation / engine

Build apps or autonomous workflows with reusable, shareable integrations connecting any service, app, blockchain or decentralized network.
https://mesg.com/
Apache License 2.0
130 stars 13 forks source link

Break internal dependencies in service #437

Closed antho1404 closed 6 years ago

antho1404 commented 6 years ago

Event, Outputs etc... have a reference on the service and the only reason they have this is to have a nice error message. The service is never needed in the actual business logic, it's just user cosmetic so I would recommend to get rid of this because it's not really needed for the business logic and it can make things complicated in the future

related PR https://github.com/mesg-foundation/core/pull/417#pullrequestreview-153245567

ilgooz commented 6 years ago

475 includes this