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

feat: dynamic address book [HIP-869] #2403

Merged
merged 7 commits into from
Aug 13, 2024
Merged

Conversation

svetoslav-nikol0v
Copy link
Contributor

@svetoslav-nikol0v svetoslav-nikol0v commented Jul 18, 2024

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

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Copy link

codecov bot commented Jul 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.45%. Comparing base (5c428d4) to head (695eb88).
Report is 91 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

@svetoslav-nikol0v svetoslav-nikol0v marked this pull request as ready for review July 18, 2024 11:51
@svetoslav-nikol0v svetoslav-nikol0v requested review from a team as code owners July 18, 2024 11:51
@svetoslav-nikol0v svetoslav-nikol0v requested review from rwalworth and removed request for rwalworth July 18, 2024 11:51
* feat: add addressBook channel

Signed-off-by: svetoslav-nikol0v <[email protected]>

* feat: implement NodeCreateTransaction class

Signed-off-by: svetoslav-nikol0v <[email protected]>

* feat: implement ServiceEndpoint class

Signed-off-by: svetoslav-nikol0v <[email protected]>

* chore: export new classess

Signed-off-by: svetoslav-nikol0v <[email protected]>

* update: proto changes

Signed-off-by: svetoslav-nikol0v <[email protected]>

* update: transaction receipt

Signed-off-by: svetoslav-nikol0v <[email protected]>

* fix: unit tests

Signed-off-by: svetoslav-nikol0v <[email protected]>

* fix: method in node create transaction

Signed-off-by: svetoslav-nikol0v <[email protected]>

---------

Signed-off-by: svetoslav-nikol0v <[email protected]>
Signed-off-by: svetoslav-nikol0v <[email protected]>
Signed-off-by: svetoslav-nikol0v <[email protected]>
Signed-off-by: svetoslav-nikol0v <[email protected]>
* feat: node update transaction

Signed-off-by: svetoslav-nikol0v <[email protected]>

* fix: adjustments

Signed-off-by: svetoslav-nikol0v <[email protected]>

---------

Signed-off-by: svetoslav-nikol0v <[email protected]>
* chore: formatting

Signed-off-by: Svet <[email protected]>

* add: example

Signed-off-by: Svet <[email protected]>

* chore: formatting

Signed-off-by: Svet <[email protected]>

* update: example

Signed-off-by: Svet <[email protected]>

---------

Signed-off-by: Svet <[email protected]>
Copy link

sonarcloud bot commented Aug 7, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
32.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Contributor

@agadzhalov agadzhalov left a comment

Choose a reason for hiding this comment

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

LGTM

@ivaylonikolov7 ivaylonikolov7 merged commit 82958b3 into main Aug 13, 2024
6 of 7 checks passed
@ivaylonikolov7 ivaylonikolov7 deleted the hip-869-dynamic-address-book branch August 13, 2024 11:37
@ivaylonikolov7 ivaylonikolov7 mentioned this pull request Aug 13, 2024
2 tasks
ivaylogarnev-limechain pushed a commit that referenced this pull request Aug 22, 2024
* feat: create node transaction (#2402)

* feat: add addressBook channel

Signed-off-by: svetoslav-nikol0v <[email protected]>

* feat: implement NodeCreateTransaction class

Signed-off-by: svetoslav-nikol0v <[email protected]>

* feat: implement ServiceEndpoint class

Signed-off-by: svetoslav-nikol0v <[email protected]>

* chore: export new classess

Signed-off-by: svetoslav-nikol0v <[email protected]>

* update: proto changes

Signed-off-by: svetoslav-nikol0v <[email protected]>

* update: transaction receipt

Signed-off-by: svetoslav-nikol0v <[email protected]>

* fix: unit tests

Signed-off-by: svetoslav-nikol0v <[email protected]>

* fix: method in node create transaction

Signed-off-by: svetoslav-nikol0v <[email protected]>

---------

Signed-off-by: svetoslav-nikol0v <[email protected]>

* fix: integration test

Signed-off-by: svetoslav-nikol0v <[email protected]>

* chore: remove .only

Signed-off-by: svetoslav-nikol0v <[email protected]>

* feat: node delete transaction (#2404)

Signed-off-by: svetoslav-nikol0v <[email protected]>

* feat: node update transaction (#2405)

* feat: node update transaction

Signed-off-by: svetoslav-nikol0v <[email protected]>

* fix: adjustments

Signed-off-by: svetoslav-nikol0v <[email protected]>

---------

Signed-off-by: svetoslav-nikol0v <[email protected]>

* update: example for HIP-869 (#2443)

* chore: formatting

Signed-off-by: Svet <[email protected]>

* add: example

Signed-off-by: Svet <[email protected]>

* chore: formatting

Signed-off-by: Svet <[email protected]>

* update: example

Signed-off-by: Svet <[email protected]>

---------

Signed-off-by: Svet <[email protected]>

* update: remove default values

Signed-off-by: Svet <[email protected]>

---------

Signed-off-by: svetoslav-nikol0v <[email protected]>
Signed-off-by: Svet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants