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: add new RPC method for block #79

Merged
merged 4 commits into from
Oct 5, 2024
Merged

Conversation

blackhorseya
Copy link
Owner

No description provided.

- Add a new RPC method `FoundNewBlockNonStream` to the BlockService
- Update input and output types for existing RPC methods to accommodate new functionality
- Modify generated gRPC client and server interfaces to include the new method
- Adjust mock implementations to support the new RPC method

Signed-off-by: Sean Zheng <[email protected]>
- Add a new method `ProcessBlockTransactionsNonStream` to the `TransactionService` for handling non-streaming block transactions.
- Update the gRPC client and server interfaces to support the new non-streaming method.
- Adjust the implementation for `ListTransactions` to reference the correct stream index.
- Modify the mock for `ProcessBlockTransactionsNonStream` to facilitate testing.
- Make corresponding changes in the protocol buffer definitions to reflect the new method.

Signed-off-by: Sean Zheng <[email protected]>
- Add a new method `FoundNewBlockNonStream` to handle found blocks
- Include a TODO note for implementation in the new method
- Add a new method `ProcessBlockTransactionsNonStream` to process transactions for blocks
- Include a TODO note for implementation in the new method

Signed-off-by: Sean Zheng <[email protected]>
@blackhorseya blackhorseya self-assigned this Oct 5, 2024
Copy link

github-actions bot commented Oct 5, 2024

Risk Level 2 - /home/runner/work/ryze/ryze/app/domain/block/block_service.go

The new method FoundNewBlockNonStream is marked with a TODO and currently panics. This should be implemented before merging to avoid runtime errors. Consider returning an error instead of panicking to improve error handling.


Risk Level 2 - /home/runner/work/ryze/ryze/app/domain/transaction/transaction_service.go

The new method ProcessBlockTransactionsNonStream is also marked with a TODO and currently panics. Similar to the previous file, this should be implemented to avoid runtime errors. Use error handling instead of panic.


🔍⚠️🛠️


Powered by Code Review GPT

Copy link

sonarcloud bot commented Oct 5, 2024

@blackhorseya blackhorseya merged commit 879dd11 into main Oct 5, 2024
4 checks passed
@blackhorseya blackhorseya deleted the feature/append-biz branch October 5, 2024 03:34
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.

1 participant