VolumeFi / paloma

The fast blockchain messenger protocol
Apache License 2.0
3 stars 2 forks source link

BUG: Curve TWAP doesn't show the token listings #862

Closed taariq closed 1 year ago

taariq commented 1 year ago

What is happening?

Section description Provide as much context as you can. Give as much context as you can to make it easier for the developers to figure what is happening.

No twap today because my coins don't show up. Only shows ETH and Fund my Bot

telegram-cloud-photo-size-1-4985698419628682280-y

How to reproduce?

Section description Please write detailed steps of what you were doing for this bug to appear.

Attempt Curve TWAP

What is the expected behaviour?

Section description If you know, please write down what is the expected behaviour. If you don't know, that's ok. We can have a discussion in comments.

All Coins show up without delay.

verabehr commented 1 year ago

this was tested ~10:49pm PT on oct 10

jamie088 commented 1 year ago

@taariq Could you please send me your wallet address? I just checked Curve TWAP on curvebot and it is working well on my side.

verabehr commented 1 year ago

@jamie088 0x7a16ff8270133f063aab6c9977183d9e72835428

verabehr commented 1 year ago

I also see all my tokens using my personal wallet btw

verabehr commented 1 year ago

closing for now as we can't reproduce reliably or see any errors in Sentry.

taariq commented 1 year ago

10/14/2023: $2MM TWAP attempt with address https://etherscan.io/address/0x7a16fF8270133F063aAb6C9977183D9e72835428

ONLY ETH tokens shows. All other tokens don't show.

Michael, [Oct 14, 2023 at 5:53:11 PM]: list is still not fixed

So I can't

**keep in mind - I am in Australia at the moment

So all the http requests are long

Might be that your thing just considers them as failed b/c it took too long bu they are not

Anyhow

Not usable ye...

telegram-cloud-photo-size-1-4999187735084378913-y

taariq commented 1 year ago

@verabehr & @jamie088 this is a Moralis issue. We are not returning the tokens list. No errors in the browser. Let's use another provider. @jamie088 please attempt to replace this with https://www.quicknode.com/token-api and see if we can get what we need.

taariq commented 1 year ago

Moralis shows that success was 100% , but no other logs

image

taariq commented 1 year ago

Okay. So maybe not moralis, but might be the Nest Wallet causing problems.

https://volume-x3.sentry.io/issues/4381900845/?alert_rule_id=13483462&alert_timestamp=1697331180177&alert_type=email&environment=production&notification_uuid=8fce0519-0368-45a7-8b1b-5f19b1aa1765&project=6380489&referrer=alert_email

This bug happened at the same time that user was attempting to TWAP.

This bug has a ticket: https://github.com/VolumeFi/paloma/issues/829

taariq commented 1 year ago

This also had a resolution: https://github.com/VolumeFi/paloma/issues/844 but it may not have been enough.

taariq commented 1 year ago

Increased the frequency in #844 to see if this is the fix.