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

Reduce rpc calls in updateSignerPenAndStatus (resolve #801) #802

Merged
merged 2 commits into from
Dec 27, 2023

Conversation

bobcoin98
Copy link
Contributor

@bobcoin98 bobcoin98 commented Dec 26, 2023

Instead of calling rpc 1 request per candidate, send a request to get status of all candidate and check

Reduce rpc calls in updateSignerPenAndStatus (resolve #801)
(This content is created via Gittool)
On behalf of bobcoin98

@bobcoin98 bobcoin98 force-pushed the updateCandidateStatus branch from 29a9f53 to 645c380 Compare December 26, 2023 06:35
Copy link
Contributor Author

@bobcoin98 bobcoin98 left a comment

Choose a reason for hiding this comment

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

This PR seems to be OK. Merging it now !
(This content is created via Gittool)
On behalf of bobcoin98

@bobcoin98 bobcoin98 merged commit 895f1b0 into master Dec 27, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve checking status of candidate process
1 participant