Skip to content

Conversation

@jaycoolslm
Copy link

@jaycoolslm jaycoolslm commented Sep 8, 2025

If you are adding a new RPC, please answer the following questions.

Link the service provider's website (the company/protocol/individual providing the RPC):

  1. https://hgraph.com

Provide a link to your privacy policy:

  1. https://hgraph.com/privacy

Your RPC should always be added at the end of the array.

@jaycoolslm
Copy link
Author

A similar PR was rejected last time with no explanation - please share comment if this is also to be rejected

@github-actions
Copy link

Validation failed for chain files. Please check the workflow logs for details.

Error message:

Validation failed for chainid-10088.js: Parent chain should start with eip155-: ethereum
Validation failed for chainid-613419.js: Parent chain should start with eip155-: ethereum

Removed validationcloud tracking details from RPC configurations.
@jaycoolslm jaycoolslm requested a review from Define101 November 25, 2025 14:30
@jaycoolslm
Copy link
Author

@Define101 please can you review?

@Define101
Copy link
Member

@jaycoolslm rpcs must be public. No api keys ${THIRDWEB_API_KEY}

Removed thirdweb RPC URLs from mainnet and testnet configurations.
@jaycoolslm
Copy link
Author

@jaycoolslm rpcs must be public. No api keys ${THIRDWEB_API_KEY}

Okay... many other examples in the file which have API keys.
Removed thirdweb, as was already accepted here #2260

LGTM

@jaycoolslm
Copy link
Author

Please re-review @Define101

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