Refactor wallet connect, automatically propose network

app/components/topbar-account-panel/component.js

@@ -1,45 +1,37 @@
-import Component from '@ember/component';
+import Component from '@glimmer/component';
+import { tracked } from '@glimmer/tracking';
 import { inject as service } from '@ember/service';
-import { computed } from '@ember/object';
+import { action } from '@ember/object';
 import { isPresent } from '@ember/utils';
 
-export default Component.extend({
+export default class TopbarAccountPanelComponent extends Component {
+  @service router;
+  @service kredits;
 
-  tagName: '',
+  @tracked setupInProgress = false;
 
-  kredits: service(),
-  router: service(),
-
-  setupInProgress: false,
-
-  userHasEthereumWallet: computed(function() {
+  get userHasWallet () {
     return isPresent(window.ethereum);
-  }),
-
-  showConnectButton: computed('userHasEthereumWallet',
-                              'kredits.hasAccounts', function() {
-    return this.userHasEthereumWallet &&
-           !this.kredits.hasAccounts;
-  }),
-
-  actions: {
-
-    signup() {
-      this.router.transitionTo('signup');
-    },
-
-    async connectAccount() {
-      try {
-        await window.ethereum.enable();
-        this.set('setupInProgress', true);
-        await this.kredits.setup();
-        this.set('setupInProgress', false);
-      } catch (error) {
-        this.set('setupInProgress', false);
-        console.log('Opening Ethereum wallet failed:', error);
-      }
-    }
-
   }
 
-});
+  get walletConnected () {
+    return this.userHasWallet && this.kredits.hasAccounts;
+  }
+
+  get walletDisconnected () {
+    return this.userHasWallet && !this.kredits.hasAccounts;
+  }
+
+  @action
+  signup () {
+    this.router.transitionTo('signup');
+  }
+
+  @action
+  async connectWallet () {
+    this.setupInProgress = true;
+    await this.kredits.connectWallet();
+    this.setupInProgress = false;
+  }
+
+}

app/components/topbar-account-panel/template.hbs

@@ -2,16 +2,15 @@
   {{#if this.setupInProgress}}
     Connecting account...
   {{else}}
-    {{#if (and this.kredits.hasAccounts this.kredits.currentUser)}}
+    {{#if (and this.walletConnected this.kredits.currentUser)}}
       {{this.kredits.currentUser.name}}
-      {{#if this.kredits.currentUserIsCore}}
-        <span class="core-flag">(core)</span>
-      {{/if}}
     {{else}}
       <span class="anonymous">Anonymous</span>
-      <button onclick={{action "signup"}} class="small" type="button">Sign up</button>
-      {{#if this.showConnectButton}}
-        <button onclick={{action "connectAccount"}} class="small green" type="button">Connect account</button>
+      <button onclick={{action "signup"}} id="signup"
+              class="small" type="button">Sign up</button>
+      {{#if this.walletDisconnected}}
+        <button onclick={{action "connectWallet"}} id="connect"
+                class="small green" type="button">Connect wallet</button>
       {{/if}}
     {{/if}}
   {{/if}}

app/services/browser-cache.js

@@ -4,8 +4,8 @@ import config from 'kredits-web/config/environment';
 
 function createStore(name) {
   let networkName;
-  if (config.web3RequiredNetworkName) {
-    networkName = config.web3RequiredNetworkName.toLocaleLowerCase().replace(' ', '-');
+  if (config.web3NetworkName) {
+    networkName = config.web3NetworkName.toLocaleLowerCase().replace(' ', '-');
   } else {
     networkName = 'custom';
   }

app/services/kredits.js

@@ -5,7 +5,7 @@ import Service from '@ember/service';
 import EmberObject from '@ember/object';
 import { computed } from '@ember/object';
 import { alias, filterBy, notEmpty, sort } from '@ember/object/computed';
-import { isEmpty, isPresent } from '@ember/utils';
+import { isEmpty } from '@ember/utils';
 import { inject as service } from '@ember/service';
 
 import { task, taskGroup } from 'ember-concurrency';
@@ -15,6 +15,7 @@ import processContributorData from 'kredits-web/utils/process-contributor-data';
 import processContributionData from 'kredits-web/utils/process-contribution-data';
 import processReimbursementData from 'kredits-web/utils/process-reimbursement-data';
 import formatKredits from 'kredits-web/utils/format-kredits';
+import switchNetwork from 'kredits-web/utils/switch-network';
 
 import config from 'kredits-web/config/environment';
 import Contributor from 'kredits-web/models/contributor';
@@ -55,6 +56,21 @@ export default Service.extend({
     this.set('contributors', []);
     this.set('contributions', []);
     this.set('reimbursements', []);
+
+    if (window.ethereum) {
+      window.ethereum.on('chainChanged', this.handleUserChainChanged);
+      window.ethereum.on('accountsChanged', this.handleAccountsChanged);
+    }
+  },
+
+  handleUserChainChanged (chainId) {
+    console.log('User-provided chain ID changed to', chainId);
+    window.location.reload();
+  },
+
+  handleAccountsChanged (accounts) {
+    console.log('User-provided accounts changed to', accounts);
+    window.location.reload();
   },
 
   // This is called in the application route's beforeModel(). So it is
@@ -64,7 +80,7 @@ export default Service.extend({
     let ethProvider;
 
     return new Promise(resolve => {
-      function instantiateWithoutAccount () {
+      function instantiateWithoutWallet () {
         console.debug('[kredits] Creating new instance from npm module class');
         console.debug(`[kredits] providerURL: ${config.web3ProviderUrl}`);
         ethProvider = new ethers.providers.JsonRpcProvider(config.web3ProviderUrl);
@@ -74,44 +90,50 @@ export default Service.extend({
         });
       }
 
-      async function instantiateWithAccount (web3Provider, context) {
+      async function instantiateWithWallet (web3Provider, context) {
         console.debug('[kredits] Using user-provided Web3 instance, e.g. from Metamask');
-        ethProvider = new ethers.providers.Web3Provider(web3Provider);
+        ethProvider       = new ethers.providers.Web3Provider(web3Provider);
+        const network     = await ethProvider.getNetwork();
+        const accounts    = await ethProvider.listAccounts();
+        const chainId     = config.web3ChainId;
 
-        const network = await ethProvider.getNetwork();
-        if (isPresent(config.web3RequiredChainId) &&
-            network.chainId !== config.web3RequiredChainId) {
-          return instantiateWithoutAccount();
-        }
+        if (isEmpty(accounts)) return instantiateWithoutWallet();
 
-        ethProvider.listAccounts().then(accounts => {
+        if (network.chainId !== chainId) {
+          return switchNetwork();
+        } else {
           context.set('currentUserAccounts', accounts);
           const ethSigner = accounts.length === 0 ? null : ethProvider.getSigner();
           resolve({
             ethProvider,
             ethSigner
           });
-        });
+        }
       }
 
       if (window.ethereum) {
-        if (window.ethereum.isConnected()) {
-          instantiateWithAccount(window.ethereum, this);
-        } else {
-          instantiateWithoutAccount();
-        }
-      }
-      // Legacy dapp browsers...
-      else if (window.web3) {
-        instantiateWithAccount(window.web3.currentProvider, this);
-      }
-      // Non-dapp browsers...
-      else {
-        instantiateWithoutAccount();
+        instantiateWithWallet(window.ethereum, this);
+      } else {
+        instantiateWithoutWallet();
       }
     });
   },
 
+  async connectWallet () {
+    const provider    = new ethers.providers.Web3Provider(window.ethereum);
+    const network     = await provider.getNetwork();
+    const chainId     = config.web3ChainId;
+    const chainIdHex  = `0x${Number(chainId).toString(16)}`;
+
+    try {
+      await window.ethereum.request({ method: 'eth_requestAccounts' });
+      if (network.chainId !== chainId) await switchNetwork(chainIdHex);
+    } catch (err) {
+      console.log('Connecting wallet failed:', err);
+      return false;
+    }
+  },
+
   async setup () {
     const kredits = await this.getEthProvider().then(providerAndSigner => {
       return new Kredits(providerAndSigner.ethProvider, providerAndSigner.ethSigner, {

app/templates/dashboard.hbs

@@ -45,14 +45,16 @@
       </div>
     </section>
 
-    <section id="contributions-by-type">
-      <header>
-        <h2>Contributions by type</h2>
-      </header>
-      <div class="content">
-        <ChartContributionsByType @contributions={{this.contributions}} />
-      </div>
-    </section>
+    {{#if this.contributionsConfirmed}}
+      <section id="contributions-by-type">
+        <header>
+          <h2>Contributions by type</h2>
+        </header>
+        <div class="content">
+          <ChartContributionsByType @contributions={{this.contributions}} />
+        </div>
+      </section>
+    {{/if}}
   </div>
 
   <div id="contributions">
@@ -87,24 +89,26 @@
       </section>
     {{/if}}
 
-    <section id="contributions-confirmed">
-      <header class="with-nav">
-        <h2>Confirmed Contributions</h2>
-        <nav>
-          <button type="button"
-                  onclick={{action "toggleQuickFilterConfirmed"}}
-                  class="small {{if this.showQuickFilterConfirmed "active"}}">
-            filter
-          </button>
-        </nav>
-      </header>
-      <div class="content">
-        <ContributionList @contributions={{this.contributionsConfirmedSorted}}
-                          @vetoContribution={{action "vetoContribution"}}
-                          @selectedContributionId={{this.selectedContributionId}}
-                          @showQuickFilter={{this.showQuickFilterConfirmed}} />
-      </div>
-    </section>
+    {{#if this.contributionsConfirmed}}
+      <section id="contributions-confirmed">
+        <header class="with-nav">
+          <h2>Confirmed Contributions</h2>
+          <nav>
+            <button type="button"
+                    onclick={{action "toggleQuickFilterConfirmed"}}
+                    class="small {{if this.showQuickFilterConfirmed "active"}}">
+              filter
+            </button>
+          </nav>
+        </header>
+        <div class="content">
+          <ContributionList @contributions={{this.contributionsConfirmedSorted}}
+                            @vetoContribution={{action "vetoContribution"}}
+                            @selectedContributionId={{this.selectedContributionId}}
+                            @showQuickFilter={{this.showQuickFilterConfirmed}} />
+        </div>
+      </section>
+    {{/if}}
 
     {{#if this.showFullContributionSync}}
     <section id="sync-all-contributions">

app/templates/signup/account.hbs

@@ -8,20 +8,21 @@
     part in the community's project governance and finances.
   </p>
   <p>
-    In order to interact with the system you will need a <a
-      href="https://rootstock.io">Rootstock</a> wallet/account.
+    In order to interact with the system you will need a
+    <a href="https://rootstock.io" target="_blank" rel="noopener noreferrer">Rootstock</a>
+    wallet/account.
   </p>
   <form>
     <p>
       <label>
-        Rootstock account:<br>
+        Rootstock address:<br>
         <Input @type="text" @value={{this.accountAddress}} placeholder="0xF18E631Ea191aE4ebE70046Fcb01a436554421BA4" class={{if this.isValidEthAccount "valid" ""}} />
       </label>
     </p>
   </form>
   <p class="actions">
-    <button disabled={{this.signupButtonDisabled}}
-            onclick={{action "completeSignup"}}
+    <button {{on "click" (fn this.completeSignup)}}
+            disabled={{this.signupButtonDisabled}}
             type="button">
       Complete my profile
     </button>

app/utils/switch-network.js

@@ -0,0 +1,29 @@
+import config from 'kredits-web/config/environment';
+
+export default async function () {
+  const networkName = config.web3NetworkName;
+  const chainId     = config.web3ChainId;
+  const chainIdHex  = `0x${Number(chainId).toString(16)}`;
+
+  try {
+    await window.ethereum.request({
+      method: 'wallet_switchEthereumChain',
+      params: [{ chainId: chainIdHex }]
+    });
+  } catch (err) {
+    // This error code indicates that the chain has not been added to MetaMask
+    if (err.code === 4902) {
+      await window.ethereum.request({
+        method: 'wallet_addEthereumChain',
+        params: [{
+          chainId: chainIdHex,
+          chainName: networkName,
+          rpcUrls: [ config.web3ProviderUrl ],
+          nativeCurrency: { name: 'tRBTC', symbol: 'tRBTC', decimals: 18 }
+        }]
+      });
+    } else {
+      console.warn('Failed to switch chains:', err.message);
+    }
+  }
+}

config/environment.js

@@ -26,8 +26,8 @@ module.exports = function(environment) {
     },
 
     web3ProviderUrl: 'https://rsk-testnet.kosmos.org',
-    web3RequiredChainId: 31,
-    web3RequiredNetworkName: 'RSK Testnet',
+    web3ChainId: 31,
+    web3NetworkName: 'RSK Testnet',
 
     githubConnectUrl: 'https://hal8000.chat.kosmos.org/kredits/signup/connect/github',
     githubSignupUrl: 'https://hal8000.chat.kosmos.org/kredits/signup/github',
@@ -82,7 +82,12 @@ module.exports = function(environment) {
 
   if (process.env.WEB3_PROVIDER_URL) {
     ENV.web3ProviderUrl = process.env.WEB3_PROVIDER_URL;
-    ENV.web3RequiredChainId = null;
+  }
+  if (process.env.WEB3_CHAIN_ID) {
+    ENV.web3ChainId = parseInt(process.env.WEB3_CHAIN_ID);
+  }
+  if (process.env.WEB3_NETWORK_NAME) {
+    ENV.web3NetworkName = process.env.WEB3_NETWORK_NAME;
   }
 
   return ENV;

package.json

@@ -17,7 +17,7 @@
     "start": "ember serve",
     "test": "npm-run-all lint:* test:*",
     "test:ember": "ember test",
-    "start:local": "WEB3_PROVIDER_URL=http://localhost:8545 ember serve",
+    "start:local": "WEB3_PROVIDER_URL=http://localhost:8545 WEB3_CHAIN_ID=1337 WEB3_NETWORK_NAME='Hardhat Devchain' ember serve",
     "build": "ember build --environment=production",
     "build-prod": "rm -rf release/* && ember build -prod --output-path release",
     "preversion": "npm test",

tests/integration/components/topbar-account-panel/component-test.js

@@ -3,37 +3,90 @@ import { setupRenderingTest } from 'ember-qunit';
 import { render } from '@ember/test-helpers';
 import hbs from 'htmlbars-inline-precompile';
 
+function stubWalletApi () {
+  window.ethereum = {
+    on: function() { return true; },
+    request: function() { return Promise.resolve(); }
+  }
+}
+
 module('Integration | Component | topbar-account-panel', function(hooks) {
   setupRenderingTest(hooks);
 
-  test('unknown user without wallet (or no permission to get wallet/account info)', async function(assert) {
+  test('Unknown user without wallet', async function(assert) {
     await render(hbs`<TopbarAccountPanel />`);
 
-    assert.ok(this.element.textContent.trim().match(/^Anonymous/));
+    assert.ok(
+      this.element.textContent.trim().match(/^Anonymous/),
+      'shows current user as anonymous'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#signup').length, 1,
+      'signup button is visible'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#connect').length, 0,
+      'connect button is not visible'
+    );
   });
 
-  test('unknown user with Ethereum wallet', async function(assert) {
+  test('Unknown user with disconnected wallet', async function(assert) {
+    stubWalletApi();
+    let service = this.owner.lookup('service:kredits');
+    service.set('currentUserAccounts', []);
+    await render(hbs`<TopbarAccountPanel />`);
+
+    assert.ok(
+      this.element.textContent.trim().match(/^Anonymous/),
+      'shows current user as anonymous'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#signup').length, 1,
+      'signup button is visible'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#connect').length, 1,
+      'connect button is visible'
+    );
+  });
+
+  test('Unknown user with connected wallet', async function(assert) {
+    stubWalletApi();
     let service = this.owner.lookup('service:kredits');
     service.set('currentUserAccounts', [{ foo: 'bar' }]);
     await render(hbs`<TopbarAccountPanel />`);
 
-    assert.ok(this.element.textContent.trim().match(/^Anonymous/));
+    assert.ok(
+      this.element.textContent.trim().match(/^Anonymous/),
+      'shows current user as anonymous'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#signup').length, 1,
+      'signup button is visible'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#connect').length, 0,
+      'connect button is not visible'
+    );
   });
 
-  test('known contributor', async function(assert) {
+  test('Known contributor', async function(assert) {
+    stubWalletApi();
     let service = this.owner.lookup('service:kredits');
     service.set('currentUserAccounts', [{ foo: 'bar' }]);
-    service.set('currentUser', {
-      name: 'Dorian Nakamoto',
-      isCore: false
-    });
+    service.set('currentUser', { name: 'Dorian Nakamoto', isCore: false });
     await render(hbs`<TopbarAccountPanel />`);
 
-    assert.dom(this.element).hasText('Dorian Nakamoto');
-
-    service.set('currentUser.isCore', true);
-    await render(hbs`<TopbarAccountPanel />`);
-
-    assert.equal(this.element.querySelectorAll('span.core-flag').length, 1);
+    assert.dom(this.element).hasText(
+      'Dorian Nakamoto', 'shows current user\'s name'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#signup').length, 0,
+      'signup button is not visible'
+    );
+    assert.equal(
+      this.element.querySelectorAll('button#connect').length, 0,
+      'connect button is not visible'
+    );
   });
 });