Emit events for chain notifications (re-posted after revert)

.circleci/config.yml

@@ -33,5 +33,8 @@ jobs:
       - run:
           name: run api tests
           command: yarn test-api
+      - run:
+          name: run event tests
+          command: yarn test-events
       - store_artifacts:
           path: coverage

.eslintrc.json

@@ -47,6 +47,11 @@
     "lines-between-class-members": 0,
     "max-classes-per-file": ["error", 5],
     "max-len": ["error", {"code": 120, "tabWidth": 2}],
-    "dot-notation": 0
+    "no-trailing-spaces": ["error"],
+    "no-useless-constructor": 0,
+    "no-empty-function": 0,
+    "import/prefer-default-export": 0,
+    "dot-notation": 0,
+    "no-lonely-if": 0
   }
 }

client/scripts/controllers/chain/substrate/account.ts

@@ -386,9 +386,16 @@ export class SubstrateAccount extends Account<SubstrateCoin> {
   public get balanceTransferFee(): Observable<SubstrateCoin> {
     if (this.chainClass === ChainClass.Edgeware) {
       // grab const tx fee on edgeware
-      return this._Chain.api.pipe(
-        map((api: ApiRx) => this._Chain.coins(api.consts.balances.transferFee as Balance))
-      );
+      return from(this._Chain.api.pipe(map((api: ApiRx) => (api.consts.balances.transferFee as Balance)))
+        .toPromise()
+        .then((txFee) => {
+          if (txFee) {
+            return Promise.resolve(this._Chain.coins(txFee));
+          } else {
+            const dummyTxFunc = (api: ApiRx) => api.tx.balances.transfer(this.address, '0');
+            return this._Chain.computeFees(this.address, dummyTxFunc);
+          }
+        }));
     } else {
       // compute fee on Kusama
       const dummyTxFunc = (api: ApiRx) => api.tx.balances.transfer(this.address, '0');

client/scripts/models/ChainEvent.ts

@@ -0,0 +1,27 @@
+import { IChainEventData } from 'events/interfaces';
+import ChainEventType from './ChainEventType';
+
+class ChainEvent {
+  public readonly id: number;
+  public readonly blockNumber: number;
+  public readonly data: IChainEventData;
+  public readonly type: ChainEventType;
+
+  constructor(id, blockNumber, data, type) {
+    this.id = id;
+    this.blockNumber = blockNumber;
+    this.data = data;
+    this.type = type;
+  }
+
+  public static fromJSON(json) {
+    return new ChainEvent(
+      json.id,
+      json.block_number,
+      json.event_data,
+      ChainEventType.fromJSON(json.ChainEventType),
+    );
+  }
+}
+
+export default ChainEvent;

client/scripts/models/ChainEventType.ts

@@ -0,0 +1,19 @@
+import { SubstrateEventKind } from 'shared/events/edgeware/types';
+
+class ChainEventType {
+  public readonly id: string;
+  public readonly chain: string;
+  public readonly eventName: SubstrateEventKind;
+
+  constructor(id, chain, eventName) {
+    this.id = id;
+    this.chain = chain;
+    this.eventName = eventName;
+  }
+
+  public static fromJSON(json) {
+    return new ChainEventType(json.id, json.chain, json.event_name);
+  }
+}
+
+export default ChainEventType;

client/scripts/models/Notification.ts

@@ -1,33 +1,45 @@
 import moment from 'moment-twitter';
 import NotificationSubscription from './NotificationSubscription';
+import ChainEvent from './ChainEvent';
 
 class Notification {
   public readonly id: number;
   public readonly data: string;
   public readonly createdAt: moment.Moment;
   public readonly subscription: NotificationSubscription;
+  public readonly chainEvent?: ChainEvent;
 
   private _isRead: boolean;
   public get isRead(): boolean {
     return this._isRead;
   }
 
-  constructor(id, data, isRead, createdAt, subscription) {
+  constructor(id, data, isRead, createdAt, subscription, chainEvent?) {
     this.id = id;
     this.data = data;
     this._isRead = isRead;
     this.createdAt = moment(createdAt);
     this.subscription = subscription;
+    this.chainEvent = chainEvent;
   }
+
   public markRead() {
     if (this._isRead) {
       throw new Error('notification already read!');
     } else {
       this._isRead = true;
     }
   }
+
   public static fromJSON(json, subscription: NotificationSubscription) {
-    return new Notification(json.id, json.notification_data, json.is_read, json.created_at, subscription);
+    return new Notification(
+      json.id,
+      json.notification_data,
+      json.is_read,
+      json.created_at,
+      subscription,
+      json.ChainEvent ? ChainEvent.fromJSON(json.ChainEvent) : undefined,
+    );
   }
 }
 

client/scripts/models/index.ts

@@ -27,6 +27,8 @@ export { default as StorageModule } from './StorageModule';
 export { default as ChainObject } from './ChainObject';
 export { default as ChainObjectQuery } from './ChainObjectQuery';
 export { default as ChainObjectVersion } from './ChainObjectVersion';
+export { default as ChainEventType } from './ChainEventType';
+export { default as ChainEvent } from './ChainEvent';
 
 export { DepositVote, BinaryVote } from './votes';
 

client/scripts/views/components/header.ts

@@ -7,12 +7,17 @@ import $ from 'jquery';
 import _ from 'lodash';
 import moment from 'moment';
 import mixpanel from 'mixpanel-browser';
+import BN from 'bn.js';
 
 import { initAppState } from 'app';
 import app, { ApiStatus } from 'state';
 import { ProposalType } from 'identifiers';
 import { featherIcon, slugify } from 'helpers';
 import { NotificationCategories } from 'types';
+
+import { formatCoin } from 'adapters/currency';
+import labelEdgewareEvent from 'events/edgeware/filters/labeler';
+
 import Substrate from 'controllers/chain/substrate/main';
 import Cosmos from 'controllers/chain/cosmos/main';
 import Edgeware from 'controllers/chain/edgeware/main';
@@ -610,36 +615,54 @@ const HeaderNotificationRow: m.Component<IHeaderNotificationRow> = {
       ]);
     };
 
-    const {
-      author,
-      createdAt,
-      notificationHeader,
-      notificationBody,
-      path,
-      pageJump
-    } = getNotificationFields(category, JSON.parse(notification.data));
-
-    return getHeaderNotificationRow(
-      author,
-      createdAt,
-      notificationHeader,
-      notificationBody,
-      path,
-      pageJump
-    );
-
-    // else if (category === NotificationCategories.NewCommunity) {
-    //   //const { created_at, proposal_id } = JSON.parse(notification.data);
-    //   //const thread = app.threads.store.getByIdentifier(proposal_id);
-    //   const community = app.activeId();
-
-    //   return getHeaderNotificationRow(
-    //     moment.utc(created_at),
-    //     null,
-    //     `New community created`,
-    //     '',
-    //     `/${community}/`);
-    // }
+    if (category === NotificationCategories.ChainEvent) {
+      if (!notification.chainEvent) {
+        throw new Error('chain event notification does not have expected data');
+      }
+      // TODO: use different labelers depending on chain
+      const chainId = notification.chainEvent.type.chain;
+      const chainName = app.config.chains.getById(chainId).name;
+      const label = labelEdgewareEvent(
+        notification.chainEvent.blockNumber,
+        chainId,
+        notification.chainEvent.data,
+      );
+      return m('li.HeaderNotificationRow', {
+        class: notification.isRead ? '' : 'active',
+        onclick: async () => {
+          const notificationArray: Notification[] = [];
+          notificationArray.push(notification);
+          app.login.notifications.markAsRead(notificationArray).then(() => m.redraw());
+          if (!label.linkUrl) return;
+          await m.route.set(label.linkUrl);
+          m.redraw.sync();
+        },
+      }, [
+        m('.comment-body', [
+          m('.comment-body-top', `${label.heading} on ${chainName}`),
+          m('.comment-body-bottom', `Block ${notification.chainEvent.blockNumber}`),
+          m('.comment-body-excerpt', label.label),
+        ]),
+      ]);
+    } else {
+      const {
+        author,
+        createdAt,
+        notificationHeader,
+        notificationBody,
+        path,
+        pageJump
+      } = getNotificationFields(category, JSON.parse(notification.data));
+
+      return getHeaderNotificationRow(
+        author,
+        createdAt,
+        notificationHeader,
+        notificationBody,
+        path,
+        pageJump
+      );
+    }
   },
 };
 
@@ -695,6 +718,7 @@ const NotificationButtons: m.Component<{ notifications }> = {
     const { notifications } = vnode.attrs;
     return m('.NotificationButtons', [
       m('.button', {
+        class: notifications.length > 0 ? '' : 'disabled',
         onclick: (e) => {
           e.preventDefault();
           if (notifications.length < 1) return;
@@ -726,9 +750,8 @@ const NotificationMenu : m.Component<{ menusOpen }> = {
         class: (unreadCount > 0 || invites.length > 0) ? 'unread-notifications' : '',
       }, unreadMessage),
     }, [
+      notifications.length > 0 && m(NotificationButtons, { notifications }),
       m(Notifications, { notifications }),
-      notifications.length > 0
-        && m(NotificationButtons, { notifications }),
     ]);
   }
 };

client/scripts/views/components/settings/send_edg_well.ts

@@ -46,16 +46,19 @@ const getBalanceTransferChecks = (
       `Transfer fee: ${formatCoin(txFee)}`
     ]);
   }
-  if (canTransfer && recipientBalance.eqn(0) && (app.chain as Substrate).chain.creationfee.gtn(0)) {
-    checks.push([
-      featherIcon('info', 14, 2, '#444'),
-      `Account creation fee: ${formatCoin((app.chain as Substrate).chain.creationfee)}`
-    ]);
+
+  const creationFee = (app.chain as Substrate).chain.creationfee;
+  if (canTransfer && recipientBalance.eqn(0) && creationFee) {
+    if (creationFee.gtn(0)) {
+      checks.push([
+        featherIcon('info', 14, 2, '#444'),
+        `Account creation fee: ${formatCoin((app.chain as Substrate).chain.creationfee)}`
+      ]);
+    }
   }
-  const resultingBalance = app.chain.chain.coins(recipientBalance.add(recipientBalance.gtn(0) ?
-                                                 amount.sub(txFee) :
-                                                 amount.sub(txFee)
-                                                  .sub((app.chain as Substrate).chain.creationfee)));
+  const resultingBalance = app.chain.chain.coins(recipientBalance.add(recipientBalance.gtn(0)
+    ? amount.sub(txFee)
+    : (creationFee) ? amount.sub(txFee).sub(creationFee) : amount.sub(txFee)));
   if (recipientBalance.eqn(0) && resultingBalance.lt((app.chain as Substrate).chain.existentialdeposit)) {
     checks.push([
       featherIcon('slash', 14, 2, '#444'),