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

adding CommScope DCL TestNet chian #970

Closed
wants to merge 3 commits into from

Conversation

jni2000
Copy link

@jni2000 jni2000 commented Jan 9, 2025

As per CSA DCL recommendation, I am adding the CommScope DCL TestNet chian into the registry.

Copy link

vercel bot commented Jan 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
keplr-chain-registry ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 11, 2025 5:22pm

@editaahn
Copy link
Collaborator

Hi, please check the chainId or the JSON file name.

Chain identifier unmatched: (expected: testnet-2.0, actual: CommScope-DCL-TestNet)

You can validate the code by yourself in your local environment, regarding the details in README.

@jni2000
Copy link
Author

jni2000 commented Jan 11, 2025

Hi, please check the chainId or the JSON file name.

Chain identifier unmatched: (expected: testnet-2.0, actual: CommScope-DCL-TestNet)

You can validate the code by yourself in your local environment, regarding the details in README.

Thanks, Rita, for clarifying. I am new to this. Sorry for not reading README carefully :). Modification made accordingly.

Copy link
Collaborator

@editaahn editaahn left a comment

Choose a reason for hiding this comment

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

Would you provide nodeProvider information? It's a required property.

@jni2000
Copy link
Author

jni2000 commented Jan 13, 2025

Sorry, no need to add our chain, we ends up using a private deployment of keplr. Thanks.

@jni2000 jni2000 closed this Jan 13, 2025
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