Skip to content

adding support to edeXa Network #1068

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

aseempm
Copy link

@aseempm aseempm commented May 14, 2025

Adding Support for edeXa Network

Testnet

chainID: 1995
RPC: https://rpc.testnet.edexa.network

Mainnet

chainID: 5424
RPC: https://rpc.edexa.network

@aseempm aseempm marked this pull request as ready for review May 14, 2025 10:42
Copilot

This comment was marked as outdated.

Copy link
Contributor

@geekbrother geekbrother left a comment

Choose a reason for hiding this comment

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

@aseempm thank you for your contribution!
Can you please address the comment and rebase on the latest master branch 🙏

@geekbrother geekbrother requested a review from chris13524 June 9, 2025 14:10
@aseempm
Copy link
Author

aseempm commented Jun 11, 2025

@aseempm thank you for your contribution! Can you please address the comment and rebase on the latest master branch 🙏

Hi @geekbrother , I m trying to rebase the code. I m facing alot of conflicts.

@aseempm aseempm reopened this Jun 11, 2025
@aseempm aseempm requested a review from geekbrother June 11, 2025 12:04
@geekbrother geekbrother requested a review from Copilot June 11, 2025 13:13
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the edeXa Network by introducing a new provider implementation, updating tests and monitoring dashboards, and extending related configurations.

  • Added a new functional test module for edeXa under tests/functional/http.
  • Created a new provider implementation in src/providers/edexa.rs with accompanying configuration in src/env/edexa.rs and updates to provider registries.
  • Updated monitoring dashboards and supported chains documentation to reflect the edeXa Mainnet and Testnet configuration.

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/functional/http/mod.rs Added module entry for edeXa tests.
tests/functional/http/edexa.rs Introduced functional tests for edeXa provider.
terraform/monitoring/dashboard.jsonnet Integrated edeXa panels into monitoring dashboards.
src/providers/mod.rs Registered the new Edexa provider and updated ProviderKind enum.
src/providers/edexa.rs Implemented the edeXa provider logic and RPC proxy implementation.
src/lib.rs Added the Edexa provider to the providers registry.
src/env/mod.rs Included edeXa module in environment configuration.
src/env/edexa.rs Defined configuration for edeXa, including supported chains.
SUPPORTED_CHAINS.md Updated documentation with supported chains for edeXa.
Comments suppressed due to low confidence (1)

src/providers/mod.rs:793

  • [nitpick] Consider standardizing the capitalization of the provider name (e.g., 'Edexa' instead of 'edeXa') for consistency across the codebase.
ProviderKind::Edexa => "edeXa",

@aseempm
Copy link
Author

aseempm commented Jun 17, 2025

Hi @geekbrother, looking forward to get merged. Thanks 🙏

@aseempm
Copy link
Author

aseempm commented Jun 25, 2025

removed the conflicts.

@aseempm
Copy link
Author

aseempm commented Jun 30, 2025

Hi @geekbrother , I have update the code to latest.

Could you please check when you get a chance? 🙏

@aseempm
Copy link
Author

aseempm commented Jul 14, 2025

Hi @geekbrother @chris13524

I have resolved the existing conflicts. Please have a look once get a chance. :)

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.

2 participants