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.

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

US-960: Reducing time to polling balances, transactions and events #56

Merged
merged 2 commits into from
Oct 24, 2022

Conversation

sleyter93
Copy link
Collaborator

RSK Explorer API is polling every 1000 ms to find new changes.
Our backend is polling every 60000 ms, that's why transactions delay up to 1 minute.

Copy link
Member

@jessgusclark jessgusclark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work! I have changed the default polling time to 10 seconds which I think is good. The server is a bit fragile so I wouldn't want to hit it too much.

@jessgusclark
Copy link
Member

jessgusclark commented Sep 28, 2022

Do Not Merge Yet! There is a bug in the frontend where it does not disconnect from the socket unless the app is completely closed. We need to fix this first so the backend doesn't poll all the time.

@jessgusclark
Copy link
Member

New release was published today distributed to the app holders. I am going to wait a day to get most devices updated and then we can merge this in

@jessgusclark jessgusclark merged commit 76d694b into develop Oct 24, 2022
sleyter93 added a commit to sleyter93/rif-wallet-services that referenced this pull request Feb 23, 2024
…sksmart#56)

* Reducing time to polling balances, transactions and events

* Change default interval to 10 seconds.

Co-authored-by: Jesse Clark <hello@developerjesse.com>
jessgusclark added a commit that referenced this pull request Apr 11, 2024
* Reducing time to polling balances, transactions and events

* Change default interval to 10 seconds.

Co-authored-by: Jesse Clark <hello@developerjesse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants