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

Burnr demo's transfers are not working #1827

Closed
wirednkod opened this issue Feb 7, 2024 · 10 comments
Closed

Burnr demo's transfers are not working #1827

wirednkod opened this issue Feb 7, 2024 · 10 comments
Assignees

Comments

@wirednkod
Copy link
Contributor

Issue Description

When trying to send to the burnr wallet in Westend, the funds never arrive (tested from polkadotJS extension);
When trying to send from the burnr wallet to another account an error appears:
Uncaught (in promise) TypeError: pt.tx.balances.transfer is not a function

Steps to reproduce the issue

Steps to reproduce the issue

  1. open burnr account with funds
  2. open polkadot js apps on a westend account with funds
  3. send from each account to the other

Describe the results you received

When trying to send to the burnr wallet in Westend, the funds never arrive (tested from polkadotJS extension);
When trying to send from the burnr wallet to another account an error appears:
Uncaught (in promise) TypeError: pt.tx.balances.transfer is not a function

Describe the results you expected

Funds should be transfered without error between accounts

Substrate-Connect version

latest

Provider

Chrome

Browser version

Version 121.0.6167.139 (Official Build) (arm64)

Additional environment details

No response

Additional information

Additional information like issue happens only occasionally or issue happens with a particular architecture or on a particular setting

Screenshots

Screenshot 2024-02-07 at 23 47 13
Screenshot 2024-02-07 at 23 47 23

@ryanleecode ryanleecode self-assigned this Feb 8, 2024
@ryanleecode
Copy link
Collaborator

was able to reproduce. looking into it.

image

@wirednkod
Copy link
Contributor Author

@ryanleecode is there a deployed solution that works?

I just tested the deployed version and it is still not working:

image

@ryanleecode ryanleecode reopened this Feb 12, 2024
@kratico
Copy link
Contributor

kratico commented Feb 28, 2024

smol-dot/smoldot#1694 might block this issue.

@wirednkod
Copy link
Contributor Author

smol-dot/smoldot#1694 is merged. Latest version of Substrate connect does not work using PAPI or polkadotJS

@kratico
Copy link
Contributor

kratico commented Mar 12, 2024

Thanks for the follow up @wirednkod
We'll release a new version of substrate-connect which will later require a new release of PJS with this new substrate-connect version.

On PAPI, Could you upload an example to https://stackblitz.com/ with the issue?

@wirednkod
Copy link
Contributor Author

Hey @kratico . I was expecting a follow up from your side.
Concerning PAPI, I tried to run the demos inside the repository - which do not seem to work

@kratico
Copy link
Contributor

kratico commented Mar 12, 2024

Once polkadot-js/api#5817 is released then we need to upgrade PJS in the Burnr project and deploy.

@wirednkod regarding PAPI I don't see any issues.
Zombienet and Playwright tests are working in CI with PAPI.
I did a local sanity test and I didn't see any issues too.
I could be something related your environment.

@wirednkod
Copy link
Contributor Author

@kratico The issues I had was with the version that I tested during this comment.
You released a new one and then mentioned that everything works fine - which confused me event more. Make sure that you mention after a new version is released in order to recheck issues

@kratico
Copy link
Contributor

kratico commented Mar 12, 2024

Sorry for the confusion @wirednkod
I thought that with this comment, it was clear that a new release was coming up.

@kratico kratico closed this as completed Mar 12, 2024
@wirednkod
Copy link
Contributor Author

It did - but next comment sounded like there is nothing still updated. Just tested Burnr and it works

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

No branches or pull requests

3 participants