syscoin / pali-wallet

The Official Syscoin browser wallet. Open source, easy to use & multichain.
http://www.paliwallet.com/
MIT License
40 stars 17 forks source link

[Issue-006] : Explorer link not automatically added in Chainlist/Other test RPCs #617

Open function0xMarki opened 2 months ago

function0xMarki commented 2 months ago

What happened?

There seems to be some problem with Chainlist and Pali Wallet importing the chains. It's strange. My guess is that the Pali Wallet import method cannot differentiate the Mainnet/Testnet chain type, but simply adds the information provided to it.

When clicking to import a Mainnet chain in Chainlist, the import is correct, and the data stored on the RPC and explorer are correct, however, the problem is observed when performing the same procedure importing testnet chains from Chainlist.

Although the panel with the "Add network" information in Pali Wallet includes the explorer correctly, when it is saved it is observed that it has not been saved, which means that you must later edit the custom RPC to add the explorer address.

Rollux Testnet image

image

Polygon ZkEVM Testnet image

image

This causes the user to get an error if they want to access any link to see their wallet/transactions because a valid explorer is not found.

Captura de pantalla 2024-09-06 a las 16 54 52 Captura de pantalla 2024-09-06 a las 16 55 18

Version

v3.0.0 QA

What platform are you using?

macOS

What browser are you using?

Brave

Is this a trezor issue?

No

Relevant log output

No response

Potolski commented 2 months ago

This problem seems to be only happening with this RPC. It might be a problem with Chainlist itself. Regardless of the scenario I believe this should be moved to the backlog once MV3 is done.

github-actions[bot] commented 1 month ago

This issue has not been updated in 14 days and is now flagged as stale. If this issue is still affecting you and in need of further review, please comment on it with an update to keep it from auto closing in 7 days.

function0xMarki commented 1 month ago

This problem seems to be only happening with this RPC. It might be a problem with Chainlist itself. Regardless of the scenario I believe this should be moved to the backlog once MV3 is done.

This issue has been reported on Discord with the team since February 2024. It was recently copied here because the latest guidelines I have state that all backlogs are to be declared on Discord and Github.

However, in relation to your statement, this issue does not only occur with Chainlist.

In all the places where, after requesting to add the testnet RPC and correctly displaying the explorer data, after pressing the add button, the explorer is ultimately not added.

github-actions[bot] commented 1 month ago

This issue has not been updated in 14 days and is now flagged as stale. If this issue is still affecting you and in need of further review, please comment on it with an update to keep it from auto closing in 7 days.

github-actions[bot] commented 4 weeks ago

This issue was automatically closed because it has been flagged as stale, and subsequently passed 7 days with no further activity from the submitter or watchers.