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.

Sign up for GitHub

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

Jump to bottom

Transfer of tokens don't work #279

Open
fzeeshan opened this issue Apr 15, 2024 · 6 comments
Open

Transfer of tokens don't work #279

fzeeshan opened this issue Apr 15, 2024 · 6 comments

Comments

@fzeeshan
Copy link
Contributor

After running a node in dev mode and then using the front-end to view the blockchain activity, if a user tries to send tokens from one account to another it fails. I have identified and submitted the fix previously.
@fzeeshan
Copy link
Contributor Author

in Transfer.js line 84 needs to be modified and method name needs to be updated. a fix has been submitted, I am assuming it is being reviewed.

fzeeshan added a commit to fzeeshan/substrate-front-end-template that referenced this issue Apr 15, 2024
@fzeeshan
Update Transfer.js
8e9c4e9 
Fixes Issue jimmychu0807#279, currently called method is transfer which has been changed in substrate to transferAllowDeath, this proposed changed will fix, token sending while using the front-end
@invent360
Copy link

This fixed my problem. Not sure why it is not being merged.
Thanks @fzeeshan

@invent360 invent360 mentioned this issue May 5, 2024
Open
@ManuGarcia89
Copy link

This fixed my problem too. thanks @fzeeshan

@fzeeshan
Copy link
Contributor Author

fzeeshan commented May 6, 2024

I'm glad. It's a simple fix but I'm not sure why they didn't merge it. I also found work around all the broken tutorials.

@cbonaco1
Copy link

cbonaco1 commented May 9, 2024

thanks for sharing @fzeeshan this fixed issues for me as well, after several hours of debugging. it's a shame when tutorials have issues such as this. below is the PR with the fix. can you add a thumbs up emoji there? i've seen that get the team's attention, the more people react to a PR.

#280

@fzeeshan fzeeshan mentioned this issue May 11, 2024
Merged
@fzeeshan
Copy link
Contributor Author

By the way, in addition to this, I have also fixed node authorization tutorial on Substrate website, I already created a pull request and submitted the changes.

DrW3RK pushed a commit that referenced this issue Jun 18, 2024
@fzeeshan
Solution of Transfer of tokens don't work #279 (#282)
b5141af 
* Update Transfer.js

* Update index.js

* Update README.md

* Update Transfer.js

* Update Transfer.js

* Update index.js

* Update README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@cbonaco1 @fzeeshan @invent360 @ManuGarcia89