Remove mentions of Ethereum from UI and code, refactor wallet connect #214

2023-01-20T09:30:21Z

raucao commented

2023-01-20 09:30:21 +00:00

(Migrated from github.com)

This removes all remaining mentions of Ethereum from both the UI and the source code, and also fixes and improves all known wallet/account connect issues.

Furthermore, it will ask the visitor to switch their connected wallet to RSK Testnet (in the app's production mode), or to add it as new custom network (using our own RSK node), and then switch, if the network ID is not available yet. Please note that this does not work properly with localhost networks in dev (at least it's broken in Metamask atm).

closes #150, #170, #215

This removes all remaining mentions of Ethereum from both the UI and the source code, and also fixes and improves all known wallet/account connect issues. Furthermore, it will ask the visitor to switch their connected wallet to RSK Testnet (in the app's production mode), or to add it as new custom network (using our own RSK node), and then switch, if the network ID is not available yet. Please note that this does not work properly with localhost networks in dev (at least it's broken in Metamask atm). closes #150, #170, #215

bumi (Migrated from github.com) reviewed 2023-01-20 09:30:21 +00:00

raucao commented

2023-02-02 04:28:20 +00:00

(Migrated from github.com)

@bumi This PR is basically finished. However, signup is actually broken, because hal8000 is not allowed to create profiles anymore. I don't remember what our proposed solution for replacing the previous permission for it was.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kredits/kredits-web#214