Open Olshansk opened 1 year ago
@RawthiL I know you've had concerns around the decision of your research impacting development, so I wanted to get your feedback on the scope of this issue before we put it into production.
Please review it and let me know if you have any questions or recommendations for change.
No questions, you are coding it compatible to any option (multi / single geozone) if I understand it correctly, which is best given that we have not reached an agreement on it. We can later force the list of geozones to one when staking if a singe-geozone logic is implemented.
No questions, you are coding it compatible to any option (multi / single geozone) if I understand it correctly, which is best given that we have not reached an agreement on it. We can later force the list of geozones to one when staking if a singe-geozone logic is implemented.
For sure. Just wanted to keep you posted and in the loop as promised per the discussion we started here.
EDIT: Currently blocked by https://github.com/pokt-network/pocket/pull/683.
Objective
Add the first iteration of GeoZones to the utility module.
Origin Document
Goals
Deliverable
GeoZone
to the entire codebase where relevant (persistence, postgres, utility, protobufs, CLI, etc...)Non-goals / Non-deliverables
General issue deliverables
Testing Methodology
make test_all
LocalNet
is still functioning correctly by following the instructions at docs/development/README.mdk8s LocalNet
is still functioning correctly by following the instructions hereCreator: @Olshansk