Skip to content
This repository has been archived by the owner on Dec 13, 2019. It is now read-only.

[node] Deposit ERC20 #1827

Merged
merged 19 commits into from
Jul 9, 2019
Merged

[node] Deposit ERC20 #1827

merged 19 commits into from
Jul 9, 2019

Conversation

cf19drofxots
Copy link
Member

@cf19drofxots cf19drofxots commented Jul 8, 2019

This adds support for depositing ERC20 tokens into a channel.

Mainly, the API for the deposit method has been extended to include an optional parameter: token address, allowing the operator to specify which ERC20 token they'd like to deposit into the channel.

export type DepositParams = {
multisigAddress: string;
amount: BigNumber;
tokenAddress?: string;
notifyCounterparty?: boolean;
};

An upcoming PR will add support with withdrawals, and another PR after that to add support for using deposited ERC20 tokens to fund apps.

@cf19drofxots cf19drofxots added 📦 Node code related to packages/node Feature labels Jul 8, 2019
@cf19drofxots cf19drofxots self-assigned this Jul 8, 2019
@cf19drofxots cf19drofxots mentioned this pull request Jul 8, 2019
5 tasks
@cf19drofxots cf19drofxots merged commit c5d8b8a into master Jul 9, 2019
@cf19drofxots cf19drofxots deleted the deposit_erc20 branch July 9, 2019 06:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature 📦 Node code related to packages/node
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants