Skip to content

Conversation

zouxyan
Copy link
Contributor

@zouxyan zouxyan commented Sep 15, 2025

Add Jovay Sepolia Testnet (chainId: 2019775) & Jovay Mainnet (chainId: 5734951)

  • Add configuration for Jovay Sepolia Testnet
  • Add configuration for incubating Jovay Mainnet
  • Native currency: ETH (18 decimals)

Copy link

You successfully submitted a PR! Due to the amount of PRs coming in: we will only look at PRs that the CI is happy with. We can also not hold your hand getting the CI green - just look how others that where merged did it and RTFM. So as long as there is any CI check that reports an error - no human will look at this. You might be able to ask for some support after supporting the project - e.g. by sending funds to lists.eth. When you fixed things after a requested change - then you also need to (re-)request a review.

@@ -0,0 +1,8 @@
[
{
"url": "ipfs://QmahPPTTPxdvff9DXiRi7mNMgWWh7FN7JhPTusML5P7Fww",
Copy link
Collaborator

Choose a reason for hiding this comment

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

This CID does not resolve for me

Copy link
Contributor Author

@zouxyan zouxyan Sep 19, 2025

Choose a reason for hiding this comment

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

hi, can we use the gateway pinned hash, like this bafkreif3xfpoojvgf23cf6kxx7q3pgbeqtkiiv3lro23i4lemb3wiodmq4 ?

I have updated the ipfs CID of Jovay icon, plz check it again. Thanks a lot. 🙏 And I tried it from my local PC, it looks alright~🫶
image

"explorers": [
{
"name": "Jovay Explorer",
"url": "https://explorer.jovay.io",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like this doesn't resolve

Copy link
Contributor Author

@zouxyan zouxyan Sep 20, 2025

Choose a reason for hiding this comment

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

Sorry, my coworker has fixed it. 🫶 Please try it again.

I tried the test from my local ⬇️

image

@FrederikBolding
Copy link
Collaborator

Needs a rebase, but I cannot do it due to fork settings

@zouxyan
Copy link
Contributor Author

zouxyan commented Sep 20, 2025

Needs a rebase, but I cannot do it due to fork settings

@FrederikBolding It has been done🫶, plz check it again. 🙏

@FrederikBolding FrederikBolding merged commit b82a3da into ethereum-lists:master Sep 20, 2025
4 checks passed
Copy link

PR merged - please consider contributing some funds to lists.eth

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