movementlabsxyz / movement

The Movement Network is a Move-based L2 on Ethereum.
Apache License 2.0
49 stars 48 forks source link

Unable to send funds from Wallets(Petra) #164

Closed andyjsbell closed 1 week ago

andyjsbell commented 2 months ago

Describe the bug Using accounts that have been funded we are unable to send funds form wallet UI to other accounts on the suzuka testnetwork.

To Reproduce Steps to reproduce the behavior:

  1. Add Suzuka Testnet to Petra Wallet
  2. Fund from Faucet
  3. Attempt to send funds to another wallet but unable to do so
  4. See error
  5. Expected behavior
  6. Funded wallet should be able to send Move to another wallet

image

andyjsbell commented 2 months ago

It seems Razor Wallet is also running into similar issues see screenshot below after funding the account it is unable to send funds to any other wallet or do any basica transaction similar to Petra

image

Rahat-ch commented 2 months ago

Additional information from nightly team having same issue

Screen Shot 2024-07-04 at 10 27 07 AM
0xPrimata commented 1 month ago

NOTE: I was checking for versioning at first thinking that it was an issue with what aptos-core we were using. The one we use in flake.nix is fairly old, got bumped on april 25th. We might want to bump it up to the latest one we have in https://github.com/movementlabs/aptos-core

re: @Rahat-ch we are currently performing the coin registration, believe that was a false report. If it's something else, glad to take a look into it.

Razor is fine sending to an initialized or uninitialized wallet Screenshot 2024-07-11 at 6 50 56 PM Petra transaction simulation errors on the same issue present on a persistent issue across the board. This error is produced whenever a request is made to the GraphQL Indexer, it doesn't receive a response and is unable to loop through the response object because the object is Not Found. image

Should be solved by https://github.com/movementlabsxyz/movement/tree/l-monninger/indexer-integration

andyjsbell commented 1 month ago

@0xPrimata @Rahat-ch the indexer will be included shortly but I don't think this will fix the issue. The indexer as it has been termed is the raw gRPC stream of transactions that are consumed by the Transaction Stream Service which is in turn consumed by the Indexer API which finally exposes the expected GraphQL API. Once we have the referenced PR merged and deployed infrastructure for the TSS and Indexer API would be needed. We have plans to have this infrastructure in place within the next two weeks.