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

Adding Transfer/Moving tokens option in Token Manager #1134

Closed
wants to merge 1 commit into from
Closed

Adding Transfer/Moving tokens option in Token Manager #1134

wants to merge 1 commit into from

Conversation

millosolomillo
Copy link

Hi,

I don't know if what I have done is correct or if I have opened the pull request at the correct place (it is my first ever pull request 😃.

Next steps would be:

  • Adding (a button?) at the front end,
  • Adding the command on the CLI.

Feel free to correct any mistake I may have done.

Thank you!

@welcome
Copy link

welcome bot commented Apr 16, 2020

Thanks for opening this pull request! Someone will review it soon 🔍

@CLAassistant
Copy link

CLAassistant commented Apr 16, 2020

CLA assistant check
All committers have signed the CLA.

@sohkai
Copy link
Contributor

sohkai commented Jun 9, 2020

Hey @millosolomillo, thanks for pushing this and sorry for taking so long to respond!

Unfortunately at this time, and due to the nature of smart contract changes (requiring on-chain upgrades), we have placed the original issue this appears to be tackling (#419) into the "upgrade wishlist".

I am closing this at this point, because we have no defined plans to do these upgrades in the near future, but can re-open these issues as we decide to work on them!

Sorry again, I know it can be frustrating to work on something to only have it be rejected. Please let me know here, via Discord, or twitter if you'd like to pick up something else that aligns with our plans!

@sohkai sohkai closed this Jun 9, 2020
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.

3 participants