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

New stATOM vault not appearing in dapp-inter FE after what looks like successful vault creation txn #263

Closed
otoole-brendan opened this issue Apr 8, 2024 · 9 comments
Closed

New stATOM vault not appearing in dapp-inter FE after what looks like successful vault creation txn #263

otoole-brendan opened this issue Apr 8, 2024 · 9 comments
Assignees
@samsiegart
Labels
bug Something isn't working

Comments

@otoole-brendan
Copy link

otoole-brendan commented Apr 8, 2024 •
edited
Loading

Describe the bug

Created stATOM vault - thought successfully in UI (got pop-up) but not shown in my vaults view. Suspect something UI/client side as:

To Reproduce

Steps to reproduce the behavior:

  1. Got app.inter.trade
  2. Create a stATOM vault (or maybe another vault not sure)
  3. stATOM vault does not appear despite txn looking like it succeeds
  4. Refreshing does nothing

Expected behavior

stATOM vault should appear

Additional context

Tried changing between KJnodes and Polkachu for RPC endpoint. KJ nodes won't load my wallet at all but Polkachu gives CORS policy error (amongst other errors)

Screenshots

image image
@otoole-brendan otoole-brendan added the bug Something isn't working label Apr 8, 2024
@turadg
Copy link
Member

turadg commented Apr 8, 2024

RPC server failure, not a code problem.

AGORIC_NET=main  agoric wallet show --from=agoric10td8wuyznhe0eq8spm3pzt65nejf6phn8vrjgh

times out with error:

agoric: (Error#1)
Error#1: RPC failure (https://agoric.rpc.kjnodes.com:443,https://agoric-agor-rpc.polkachu.com:443): cannot read data of published.agoricNames.brand: invalid json response body at https://agoric.rpc.kjnodes.com/abci_query?path=%22/custom/vstorage/data/published.agoricNames.brand%22&height=0 reason: Unexpected token '<', "<html><bod"... is not valid JSON

because that abci_query times out.

I'll leave this open in case PM wants the solution to include better user feedback in this event.

@erights
Copy link
Member

erights commented Apr 8, 2024

Note that the underlying cause is fixed at pinojs/pino#1939 . Would adopting that difference as a (simple!) patch work in the meantime, until we can upgrade to depend on a pino with that fixed?

Attn @samsiegart @kriskowal

@turadg
Copy link
Member

turadg commented Apr 8, 2024

@erights I think you're speaking to some of the errors in the console, but this is about the UX.

We still do need to solve the console output, which is in progress: #264

@erights
Copy link
Member

erights commented Apr 8, 2024

Aha #264 !

Thanks @0xpatrickdev !

@erights
Copy link
Member

erights commented Apr 8, 2024

@erights I think you're speaking to some of the errors in the console, but this is about the UX.

We still do need to solve the console output, which is in progress: #264

Yes, I'm reacting to the console output.

@otoole-brendan
Copy link
Author

For when you're back @samsiegart - I'm keen to understand in this scenario why we didn't show the user the connection issue dialog (like we do when an RPC connection breaks/is flaky) - which directs them to change RPC/API endpoint via the connection settings?
(we should be able to know and alert users appropriately if it's connection/rpc related in the UI and not rely on them figuring it out via console logging)

@samsiegart
Copy link
Collaborator

The CORS error is not coming from the Agoric RPC, the logs show it's coming from coingecko API. which is probably being fetched from the leap elements IST onboarding modal. Are you still able to reproduce this @otoole-brendan? I think we had RPC issues last week? It could just be that the RPC was not giving the latest data, or taking a really long time to update. As you said we do have retry logic that will show the error dialog after a few unsuccessful attempts, and nothing from the logs indicates we got an error response from the RPC, so I'm not inclined to think that's the part that's broken.

@otoole-brendan
Copy link
Author

The CORS error is not coming from the Agoric RPC, the logs show it's coming from coingecko API. which is probably being fetched from the leap elements IST onboarding modal. Are you still able to reproduce this @otoole-brendan? I think we had RPC issues last week? It could just be that the RPC was not giving the latest data, or taking a really long time to update. As you said we do have retry logic that will show the error dialog after a few unsuccessful attempts, and nothing from the logs indicates we got an error response from the RPC, so I'm not inclined to think that's the part that's broken.

No it was resolved and I cannot reproduce anymore. So sounds like we can chalk it up to those RPC issues and close?

@samsiegart
Copy link
Collaborator

No it was resolved and I cannot reproduce anymore. So sounds like we can chalk it up to those RPC issues and close?

Yea, can't say 100% for sure without reproing but I'll close and we can reopen if it pops up again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samsiegart samsiegart

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@turadg @erights @samsiegart @otoole-brendan