Description: Following instruction from MetaMask support, we’ve submitted two PRs to add Wanchain network support on both Metmask Extension and MetaMask Mobile. The PRs require review from the MetaMask team. MetaMask support subsequently instructed us to post here as a feature request.
Purpose: To add Wanchain network support including chain ID, native currency (WAN), and network logo
Extension/Mobile/Both: Both, but extension is more important.
Images/Attachments:
PR1: MetaMask/MetaMask-extension/pull/40391
PR2: MetaMask/MetaMask-mobile/pull/26540
HI @WanchainDev , could you provide a bit more context into what are you refering to as network support ?
Are there any technical/ connectivity issues with the network or is this post about wanting to add the network to the networks list in MetaMask?
Hi @Coffee7643 , thanks for the prompt reply.
While having Wanchain appear in the network list in MetaMask would of course be ideal, we understand this is not realistic and fine with needing to add Wanchain as a custom network. However, once added as a Custom network, we are hoping for the network logo to appear.
My pleasure, here to help!
I understand now, thanks a lot for providing the additional context. Let me check internally.
1 Like
The team’s confirmed that the two PR approach is the correct one!
If you’ve completed the requests correctly the team will get back to you via them as soon as they are able to, feel free to let me know if you do have any other questions regarding this!
Here are two example PRs from the past for mobile and extension if you want to cross check them with your own:
main ← fix/add-mode-logo
opened 10:56AM - 23 Jan 25 UTC
## **Description**
[](https://codespaces.new/MetaMask/metamask-extension/pull/29870?quickstart=1)
## **Related issues**
Fixes:
## **Manual testing steps**
1. add mode network and mode testnet network
2. check the logo
## **Screenshots/Recordings**
### **Before**
### **After**
<img width="350" alt="Screenshot 2025-01-23 at 11 55 34" src="https://github.com/user-attachments/assets/b05be061-fda2-41ed-a250-1738333fab34" />
## **Pre-merge author checklist**
- [ ] I've followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Extension Coding Standards](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've completed the PR template to the best of my ability
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors.
## **Pre-merge reviewer checklist**
- [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.
main ← fix/add-kaia-logo
opened 11:14AM - 08 Jan 25 UTC
## **Description**
add logo for Kaia network
## **Related issues**
… Fixes:
## **Manual testing steps**
1. Go to this page...
2.
3.
## **Screenshots/Recordings**
### **Before**
### **After**
<img width="433" alt="Screenshot 2025-01-08 at 12 13 06" src="https://github.com/user-attachments/assets/e61f4806-94b2-43f5-9ce6-4a29dd2066e9" />
## **Pre-merge author checklist**
- [ ] I’ve followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile Coding Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've completed the PR template to the best of my ability
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors.
## **Pre-merge reviewer checklist**
- [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.