Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(facade): EthAnchor integration #8

Open
wants to merge 36 commits into
base: dev
Choose a base branch
from

Conversation

byeongsu-hong
Copy link

@byeongsu-hong byeongsu-hong commented Jul 19, 2021

👀 What is Changed?

  • facade
    • added ether-anchor-earn
    • use generics to AnchorEarn superclass
    • moved CHAINS, NETWORKS enums to ./src/types.ts
  • facade/interface
    • added ethereum contract definitions / interface

☑️ Checklist

  • The code compiles successfully
  • I've run yarn lint to lint and check code style
  • I've written the test code and runs successfully

MSNTCS and others added 30 commits June 18, 2021 19:19
* axios - for request to eth anchor api
* ethers
@byeongsu-hong byeongsu-hong changed the base branch from master to dev July 22, 2021 03:53
@byeongsu-hong byeongsu-hong marked this pull request as ready for review July 22, 2021 03:53
@MSNTCS MSNTCS self-requested a review July 22, 2021 04:10

const signedTx = await signer(tx);
const txResp = await this._provider.sendTransaction(signedTx);
logger(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a stateful logger so that a user can track the current state of the tx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants