celo-org / celo-monorepo

Official repository for core projects comprising the Celo platform
https://celo.org
Apache License 2.0
684 stars 360 forks source link

Notification service uses Contract Kit #225

Closed jmrossy closed 4 years ago

jmrossy commented 4 years ago

Expected Behavior

  1. Remove the config files
  2. Make service get all addresses it needs from the contract kit
  3. Move the remaining configs to the yaml files like blockchain api has

Current Behavior

Addresses are specified statically in the env configs

sallyjyl commented 4 years ago

Made some changes to the contract kit such that addresses for gold token and stable tokens and etc, can be accessed without having to get the contracts from web3 just for that purpose. Notification services use those addresses, and without those changes notification service would need to dip into the mobile package (where web3 resides currently) to use the addresses in notification services.

Facing the following issue in trying to test by deploying: Couldn't get npm to install @celo/contractkit from git+https://github.com/sallyjyl/contractKit-test.git due tono such file or directory, chmod '/Users/sally/celo/celo-monorepo/packages/notification-service/node_modules/@celo/contractkit/bin/build-sdk.js. The error is reproduceable by running: npm install or npm install git+https://github.com/sallyjyl/contractKit-test.git Work branch: sallyjyl/notif-contract-kit

Here are some of the things I've tried:

Observations:

Because it's always returning the same error about build-sdk.js, not sure where the bug is coming from.