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 <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>
@svetoslav-nikol0v svetoslav-nikol0v force-pushed the hip-869-dynamic-address-book branch from 056f10f to 516b7f3 Compare August 6, 2024 16:20
Signed-off-by: Svet <svetoslav.nikolov@limechain.tech>
Copy link

sonarqubecloud 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 <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>
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