Skip to content

Add Firo RPC scanner implementation#7

Merged
vorujack merged 6 commits intorosen-bridge:devfrom
navidR:feature/dev-aleflm-add-firo
Feb 14, 2026
Merged

Add Firo RPC scanner implementation#7
vorujack merged 6 commits intorosen-bridge:devfrom
navidR:feature/dev-aleflm-add-firo

Conversation

@navidR
Copy link
Contributor

@navidR navidR commented Jan 25, 2026

This pull request introduces the @rosen-bridge/firo-rpc-scanner package, designed to facilitate blockchain monitoring for the Firo network via JSON-RPC interactions. It includes the implementation of the FiroRpcNetworklayer for retrieving block and transaction data, the FiroRpcScanner class for integrating with the general abstract scanner architecture, and comprehensive type definitions and unit tests to ensure robust data processing.

@navidR navidR force-pushed the feature/dev-aleflm-add-firo branch 2 times, most recently from b85d50a to 5b331d4 Compare January 29, 2026 22:15
@navidR navidR changed the title add firo scanner Add Firo RPC scanner implementation Jan 30, 2026
@navidR navidR marked this pull request as ready for review January 30, 2026 07:27
@navidR
Copy link
Contributor Author

navidR commented Feb 2, 2026

@RaaCT0R Thanks for the review. All the raised concerns addressed in a11b459

@navidR navidR force-pushed the feature/dev-aleflm-add-firo branch from a11b459 to affecac Compare February 9, 2026 09:20
@navidR
Copy link
Contributor Author

navidR commented Feb 9, 2026

No content change. Just rebased to the latest dev to resolve the conflict.

@navidR
Copy link
Contributor Author

navidR commented Feb 11, 2026

Thanks for the review, @RaaCT0R. I have addressed all the issues raised.

Copy link
Contributor

@vorujack vorujack left a comment

Choose a reason for hiding this comment

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

Also please sync with base branch

@navidR navidR force-pushed the feature/dev-aleflm-add-firo branch from 7736a87 to 59f381c Compare February 14, 2026 09:22
@navidR
Copy link
Contributor Author

navidR commented Feb 14, 2026

Also please sync with base branch

Rebased to the latest master: 8bfa13e

@navidR navidR force-pushed the feature/dev-aleflm-add-firo branch from 8bfa13e to c1ad944 Compare February 14, 2026 10:32
@vorujack vorujack merged commit ffeec8a into rosen-bridge:dev Feb 14, 2026
1 check passed
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.

3 participants