Skip to content

Conversation

@manishdait
Copy link
Contributor

This PR introduces the NetworkRepository interface and its implementation to interact with the Mirror Node REST API to query network related information.

Closes: #38

- Added records for ExchangeRate, NetworkFee, NetworkStake and NetworkSupplies

- Implemented method in MirroNodeClient to query exchangeRate, fees, stake and supplies

- Added test for NetworkRepository

Signed-off-by: Manish Dait <daitmanish88@gmail.com>
@manishdait
Copy link
Contributor Author

Hi, @hendrikebbers, findNetworkSupplies() test is working on intellij but failing in CI.
Screenshot from 2024-11-12 18-26-56

@hendrikebbers
Copy link
Member

@manishdait this is a really great addition. Thank you so much :) In CI, the tests are executed against a small networkub.com/OpenElements/hedera-solo-action. that is created by https://github.com/OpenElements/hedera-solo-action. Maybe it does not support the suplies functionality. Can you deactive the unit test for now and create an issue that the test should be fixed for CI in future. Once that is done I will merge your great work!

Signed-off-by: Manish Dait <daitmanish88@gmail.com>
@manishdait
Copy link
Contributor Author

@hendrikebbers, I have disabled findNetworkSupplies() for NetworkRepositoryTest.

@hendrikebbers
Copy link
Member

Great. CI build is still broken but that belongs on an error done by me.

@hendrikebbers hendrikebbers merged commit 605153f into OpenElements:main Nov 14, 2024
2 of 3 checks passed
@manishdait manishdait deleted the 38-issue branch November 15, 2024 08:00
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.

Create a repository for network information

2 participants