-
-
Notifications
You must be signed in to change notification settings - Fork 929
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
data is never updated if view method reverts #3933
Comments
I tried to make a new reproductionable example on https://new.wagmi.sh/ but no matter what I do the page is rendered blank. Tried the latest OPERA and latest CHROME both ended up with a blank page. |
Can you share the contract address and ABI so someone that helps doesn't need to manually type the address and look for the ABI? |
I believe deploying this contract on a test net / local network would be a little to complicated, so would be better to write a simple contract with a parameter that triggers the failed assert. |
This is a silly thing but try using VPN, or different internet connections as well. |
Describe the bug
Problem
I'm using useReadContract in order to have live updates from a variable called
account_debt
, in order to fetch that data I read a "view" method on a smart contract calledget_pending_market_state_for_account
.The way the view functions works is:
account_debt
The natural user flow on the application goes as this:
The problem is when a user pays the debt on step (3) the UI never returns to (1), because:
account_debt
is nullnull
because ( I assume ) Wagmi won't update the data if the contract starts reverting with the same params?Debugging
When I look at the ReactQueryDevTools on the stuck page I can see this and it NEVER changes even though I see the "fetch" happening often, the cached data is always here.
If I press the blue button "Trigger Loading" then the data is updated and becomes "null" again ( which is the original state ).
Questions
Link to Minimal Reproducible Example
https://stackblitz.com/edit/new-wagmi?file=src%2FApp.tsx
Steps To Reproduce
Described on the preview comment
Wagmi Version
2.8.1
Viem Version
2.10.2
TypeScript Version
5.4.5
Check existing issues
Anything else?
To reproduce the error I would need to send some test tokens on FTM + FTM for gas and I think it's not needed in this case as the problem is well explained.
Thanks a lot for your hard work on the library!
The text was updated successfully, but these errors were encountered: