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: added NodeGetInfoResponse to response.proto #355

Merged
merged 1 commit into from
May 22, 2024
Merged

Conversation

iwsimon
Copy link
Contributor

@iwsimon iwsimon commented May 21, 2024

Description:

added NodeGetInfoResponse to response.proto

fix #356

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>
@iwsimon iwsimon self-assigned this May 21, 2024
@iwsimon iwsimon requested review from a team as code owners May 21, 2024 21:14
Copy link
Contributor

@povolev15 povolev15 left a comment

Choose a reason for hiding this comment

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

LGTM - TY @iwsimon .
can you add the ticket number to this PR?

@iwsimon
Copy link
Contributor Author

iwsimon commented May 22, 2024

LGTM - TY @iwsimon . can you add the ticket number to this PR?

Added issue #356

Copy link

@jnels124 jnels124 left a comment

Choose a reason for hiding this comment

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

LGTM

@iwsimon iwsimon merged commit 43d9159 into main May 22, 2024
1 check passed
@iwsimon iwsimon deleted the dab-add-response branch May 22, 2024 20:09
iwsimon added a commit that referenced this pull request May 24, 2024
iwsimon added a commit that referenced this pull request May 29, 2024
….51 (#358)

* Revert "feat: added NodeGetInfoResponse to response.proto (#355)"

This reverts commit 43d9159.

* Revert "Added NodeGetInfo (#347)"

This reverts commit 17fb148.

* Revert "feat: Dynamic Address Book Phase 2 protobufs changes (#344)"

This reverts commit e650477 as hedera-servives #13265 asked.

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>

* Align the code.

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>

* Changed INVALID_MAX_AUTO_ASSOCIATIONS back to 346 by review request.

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>

---------

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>
netopyr added a commit that referenced this pull request Jul 18, 2024
* main: (21 commits)
  feat: 13135 Added PlatformState protobuf representation (#349)
  feat: Change AddressBookService package to proto (#386)
  feat: add admin_key to node_create.proto, node_update.proto and node.proto (#380)
  HIP 904 proto updates for token reject (#367)
  chore: Updated codeowners to include devops-ci (#384)
  feat: remove node_get_info.proto (#378)
  fixed INVALID_GOSSIP_CA_CERTIFICATE (#375)
  fix: Fixing single-letter mismatch to expectations (#373)
  feat: Update specification text for address book service (#346)
  feat: Add more response codes for Dynamic Address Book Phase 2 (#370)
  feat: maxAutoAssociations updates for contract transactions (#359)
  Added more response codes for Dynamic Address Book Phase 2 (#364)
  feat: Add back Dynamic Address Book Phase 2 protobufs change (#361)
  feat: Back out protobuf-changes to dynamic address book for release 0.51 (#358)
  feat: added NodeGetInfoResponse to response.proto (#355)
  feat: HIP-904 maxAutoAssociations updates (#348)
  chore: added consensus data protobuf (#350)
  Added NodeGetInfo (#347)
  feat: Dynamic Address Book Phase 2 protobufs changes (#344)
  feat: define event protobufs (#338)
  ...

# Conflicts:
#	services/basic_types.proto
#	services/schedulable_transaction_body.proto
#	services/token_service.proto
#	services/transaction_body.proto
#	services/transaction_receipt.proto
netopyr added a commit that referenced this pull request Jul 18, 2024
* main: (21 commits)
  feat: 13135 Added PlatformState protobuf representation (#349)
  feat: Change AddressBookService package to proto (#386)
  feat: add admin_key to node_create.proto, node_update.proto and node.proto (#380)
  HIP 904 proto updates for token reject (#367)
  chore: Updated codeowners to include devops-ci (#384)
  feat: remove node_get_info.proto (#378)
  fixed INVALID_GOSSIP_CA_CERTIFICATE (#375)
  fix: Fixing single-letter mismatch to expectations (#373)
  feat: Update specification text for address book service (#346)
  feat: Add more response codes for Dynamic Address Book Phase 2 (#370)
  feat: maxAutoAssociations updates for contract transactions (#359)
  Added more response codes for Dynamic Address Book Phase 2 (#364)
  feat: Add back Dynamic Address Book Phase 2 protobufs change (#361)
  feat: Back out protobuf-changes to dynamic address book for release 0.51 (#358)
  feat: added NodeGetInfoResponse to response.proto (#355)
  feat: HIP-904 maxAutoAssociations updates (#348)
  chore: added consensus data protobuf (#350)
  Added NodeGetInfo (#347)
  feat: Dynamic Address Book Phase 2 protobufs changes (#344)
  feat: define event protobufs (#338)
  ...
vtronkov pushed a commit that referenced this pull request Jul 23, 2024
added NodeGetInfoResponse to response.proto

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>
Signed-off-by: Valentin Tronkov <99957253+vtronkov@users.noreply.github.com>
vtronkov pushed a commit that referenced this pull request Jul 23, 2024
….51 (#358)

* Revert "feat: added NodeGetInfoResponse to response.proto (#355)"

This reverts commit 43d9159.

* Revert "Added NodeGetInfo (#347)"

This reverts commit 17fb148.

* Revert "feat: Dynamic Address Book Phase 2 protobufs changes (#344)"

This reverts commit e650477 as hedera-servives #13265 asked.

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>

* Align the code.

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>

* Changed INVALID_MAX_AUTO_ASSOCIATIONS back to 346 by review request.

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>

---------

Signed-off-by: Iris Simon <iris.simon@swirldslabs.com>
Signed-off-by: Valentin Tronkov <99957253+vtronkov@users.noreply.github.com>
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.

Add NodeGetInfoResponse to response.proto
4 participants