-
Notifications
You must be signed in to change notification settings - Fork 148
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
feat: dynamic address book [HIP-869] #2403
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2403 +/- ##
==========================================
+ Coverage 76.49% 84.45% +7.95%
==========================================
Files 304 308 +4
Lines 75800 77354 +1554
==========================================
+ Hits 57986 65330 +7344
+ Misses 17814 12024 -5790 ☔ View full report in Codecov by Sentry. |
* feat: add addressBook channel Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * feat: implement NodeCreateTransaction class Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * feat: implement ServiceEndpoint class Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * chore: export new classess Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * update: proto changes Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * update: transaction receipt Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * fix: unit tests Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * fix: method in node create transaction Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> --------- Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech>
Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech>
Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech>
Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech>
* feat: node update transaction Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * fix: adjustments Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> --------- Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech>
* chore: formatting Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> * add: example Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> * chore: formatting Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> * update: example Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> --------- Signed-off-by: Svet <svetoslav.nikolov@limechain.tech>
056f10f
to
516b7f3
Compare
Signed-off-by: Svet <svetoslav.nikolov@limechain.tech>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* feat: create node transaction (#2402) * feat: add addressBook channel Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * feat: implement NodeCreateTransaction class Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * feat: implement ServiceEndpoint class Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * chore: export new classess Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * update: proto changes Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * update: transaction receipt Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * fix: unit tests Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * fix: method in node create transaction Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> --------- Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * fix: integration test Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * chore: remove .only Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * feat: node delete transaction (#2404) Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * feat: node update transaction (#2405) * feat: node update transaction Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * fix: adjustments Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> --------- Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> * update: example for HIP-869 (#2443) * chore: formatting Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> * add: example Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> * chore: formatting Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> * update: example Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> --------- Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> * update: remove default values Signed-off-by: Svet <svetoslav.nikolov@limechain.tech> --------- Signed-off-by: svetoslav-nikol0v <svetoslav.nikolov@limechain.tech> Signed-off-by: Svet <svetoslav.nikolov@limechain.tech>
Description:
This HIP-869 introduces the ability to manage the Address Book via HAPI endpoints within current Address Book implementation (config.txt, file 0.0.101,& file 0.0.102). In this phase HAPI endpoints are used to facilitate changes to the Address Book, however those changes are applied through existing, monthly upgrade process.
Related issue(s):
#2335 #2336 #2337 #2338 #2339 #2406 #2407
Checklist