search

Fayyr-Organization / fayyr-beta

Instructions and feedback for the beta test of the Fayyr website.
1 stars 2 forks source link

Bug: Salles state on cards not updating #16

Closed nodra-vr closed 1 week ago

nodra-vr commented 2 years ago

I have noticed this both after listing and unlisting an item from the market. (happed twice while listing, always unlisting) It seems that in some cases data is not updated as expected. A page reload is needed to update the data after the wallet redirects the user back to the website.

See video: unlist item > wallet > listing (bugged) > refresh > listing (updated) Fast > https://drive.google.com/file/d/1szmgswZngSucD5-Mna1kWXmyy4oHPBiz/view

Using: Brave Browser [Version 1.36.119 Chromium: 99.0.4844.83 (Official Build) (64-bit)]

WAEliasmith commented 2 years ago

Nice video, that makes it very clear. Our database gets updated through the blockchain, so need a client-side method to make things look nicer when this happens for sure. Added this to the backlog.

nodra-vr commented 2 years ago

I know the feeling, I do a client-side check when it detects the tx id that gets added by the wallet redirect.

BenKurrek commented 2 years ago

I know the feeling, I do a client-side check when it detects the tx id that gets added by the wallet redirect.

We have client side code that checks if the txn is successful (thus sending out the transaction successful modal) and we'll expand this to modify state to show the result of the txn on the frontend whilst the indexer catches up and updates our DB.