Skip to content

Incremental polling for voters, rex balances & delegations#25

Draft
tomtomcrypto wants to merge 42 commits intomasterfrom
incremental
Draft

Incremental polling for voters, rex balances & delegations#25
tomtomcrypto wants to merge 42 commits intomasterfrom
incremental

Conversation

@tomtomcrypto
Copy link

@tomtomcrypto tomtomcrypto commented Jan 28, 2023

Fixes

Updates dependencies (incl. resolving all dependabot alerts)

Description

Implements incremental pooling for voters, rex balances & delegations as well as API improvements

Test scenarios

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have cleaned up the code in the areas my change touches
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings
  • I have removed any unnecessary console messages

@karynemayer karynemayer linked an issue Jan 31, 2023 that may be closed by this pull request
@tomtomcrypto tomtomcrypto linked an issue Feb 3, 2023 that may be closed by this pull request
2 tasks
@tomtomcrypto tomtomcrypto marked this pull request as ready for review February 4, 2023 16:51
Copy link
Member

@poplexity poplexity left a comment

Choose a reason for hiding this comment

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

This looks really good, much improved 🙏

Left some comments mostly around parameterizing things.

@tomtomcrypto
Copy link
Author

This looks really good, much improved 🙏

Left some comments mostly around parameterizing things.

Yeah I started doing it actually and noticed the .env VS config.js too

@karynemayer karynemayer marked this pull request as draft March 2, 2023 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment