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.

Sign up for GitHub

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

Jump to bottom

feat: query mirror node for AccountBalances, AccountInfo, ContractInfo #343

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

feat: query mirror node for AccountBalances, AccountInfo, ContractInfo #343

wants to merge 18 commits into from

Conversation

RickyLB
Copy link
Contributor

@RickyLB RickyLB commented May 29, 2024 •
edited
Loading

Signed-off-by: Ricky Saechao ricky@launchbadge.com
Description:

  • Add TokenRelationship
  • Query Mirror Node for AccountBalance and TokenRelationship and map query responses to ContractInfoQuery, AccountInfoQuery.

Related issue(s):

Fixes #

Notes for reviewer:

Checklist

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

@RickyLB RickyLB requested review from a team as code owners May 29, 2024 14:42
@RickyLB RickyLB force-pushed the feat/mirror_queries branch 2 times, most recently from 35ae8b5 to caa0da2 Compare June 3, 2024 18:37
RickyLB added 15 commits June 3, 2024 11:37
@RickyLB
chore: update protobufs
dfab82e 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
feat: add mirror node service, gateway, and router,
89dc2d9 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
feat: add mirror node service, gateway, and router,
042bb21 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
chore: format
22342ba 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
feat: add token relationship
d00e87f 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
feat: add extra context for mirrorNetworkNode and LedgerId
d11ac31 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
chore: update protobufs
5060e80 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
chore: update protobufs
dc384b6 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
feat: modify responses to use tokenRelationships
86f519b 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
fix: correct formatting for api url
0b19af3 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
feat: add contract id population test
0e69fbb 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
feat: remove fatal errors
6c20f01 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
fix: TokenBalance and TokenRelationship query response mapping
1637bab 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
test: uncomment account balance e2e test
8944b55 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
chore: update test snapshots and pre check status
10fb994 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB RickyLB requested a review from izik1 June 3, 2024 18:43
izik1
izik1 requested changes Jun 4, 2024
View reviewed changes
Sources/Hedera/MirrorNodeGateway.swift Outdated Show resolved Hide resolved
Sources/Hedera/MirrorNodeGateway.swift Show resolved Hide resolved
Sources/Hedera/MirrorNodeGateway.swift Outdated Show resolved Hide resolved
Sources/Hedera/MirrorNodeRouter.swift Outdated Show resolved Hide resolved
Sources/Hedera/MirrorNodeRouter.swift Outdated Show resolved Hide resolved
Sources/Hedera/Contract/ContractInfo.swift Outdated Show resolved Hide resolved
Sources/Hedera/Account/AccountInfo.swift Outdated Show resolved Hide resolved
Sources/Hedera/Contract/ContractInfo.swift Outdated Show resolved Hide resolved
Sources/Hedera/Contract/ContractInfoQuery.swift Outdated Show resolved Hide resolved
Sources/Hedera/Contract/ContractInfoQuery.swift Outdated Show resolved Hide resolved
izik1
izik1 previously approved these changes Jun 5, 2024
View reviewed changes
@RickyLB
fix: requested changes
7a424d8 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
chore: fmt
83cc8e2 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@RickyLB
fix: reuse http client for mirror node queries
768ff36 
Signed-off-by: Ricky Saechao <ricky@launchbadge.com>
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jun 5, 2024

Please retry analysis of this Pull-Request directly on SonarCloud

@RickyLB RickyLB requested a review from izik1 June 6, 2024 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomzhenghedera tomzhenghedera Awaiting requested review from tomzhenghedera tomzhenghedera is a code owner automatically assigned from hashgraph/release-engineering

@jeromy-cannon jeromy-cannon Awaiting requested review from jeromy-cannon jeromy-cannon is a code owner automatically assigned from hashgraph/release-engineering-managers

@rwalworth rwalworth Awaiting requested review from rwalworth rwalworth is a code owner automatically assigned from hashgraph/hedera-sdk

@izik1 izik1 Awaiting requested review from izik1

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RickyLB @izik1