nucypher / nucypher-porter

Gateway to TACo nodes of the threshold network
GNU Affero General Public License v3.0
2 stars 8 forks source link

`/get_ursulas` should perform node sampling using `TACoChildApplication` instead of `TACoApplication` #55

Closed derekpierre closed 10 months ago

derekpierre commented 10 months ago

Before merging update the nucypher dependency to not point to the branch used for PR #3341 development.

Type of PR:

Required reviews:

What this does: Related to:

Depends on:

Until we have better syncing between root and child applications, TACoChildApplication has the most up-to-date information for stakers , and so should be used for sampling nodes instead.

NOTE: once these child sampling PRs merged, TACo contracts need to be redeployed on testnet to the latest in nucypher-contracts; ideally only lynx to start, since redeployment implies new rituals need to be created because Coordinator was not previously updateable.

Issues fixed/closed:

  • Fixes #...

Why it's needed:

Explain how this PR fits in the greater context of the NuCypher Network. E.g., if this PR addresses a nucypher/productdev issue, let reviewers know!

Notes for reviewers:

What should reviewers focus on? Is there a particular commit/function/section of your PR that requires more attention from reviewers?