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

Token manager frontend (Adding move tokens in contract and in front end) #1136

Closed
wants to merge 2 commits into from
Closed

Token manager frontend (Adding move tokens in contract and in front end) #1136

wants to merge 2 commits into from

Conversation

millosolomillo
Copy link

@millosolomillo millosolomillo commented Apr 25, 2020

Hi,

This is a work in progress. I don't know if it is correct and honestly I am a little bit lost on how to continue.

I am trying to add the move tokens functionality in the token manager front end.

I already modified the contract.

Related to #1134

@millosolomillo millosolomillo changed the title Token manager frontend (Adding mvoe tokens in contract and in front end) Token manager frontend (Adding move tokens in contract and in front end) Apr 25, 2020
@sohkai
Copy link
Contributor

sohkai commented Jun 9, 2020

Hi @millosolomillo, the frontend issue for transfer tokens from the frontend (#1005) was meant to use the api.external() API with a token address, rather than be implemented directly in the contracts.

Please let me know if you'd like to still pick this up!

@sohkai
Copy link
Contributor

sohkai commented Jul 12, 2020

Hey @millosolomillo, given that the original issue (#1005) was meant to not change the contracts, I'll take the liberty of closing this now.

Please contact us on Discord if you'd like to tackle something else—we'll point you to something relevant!

@sohkai sohkai closed this Jul 12, 2020
@millosolomillo millosolomillo deleted the token-manager-frontend branch October 2, 2020 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants