From 65678a1521fe4a3907fec6fc6173726148e6b22e Mon Sep 17 00:00:00 2001
From: Enrico Fasoli <enrico.fasoli1996@gmail.com>
Date: Thu, 8 Feb 2018 23:52:06 +0100
Subject: [PATCH] huge usability improvements

---
 package.json                |  1 +
 src/components/Board.js     | 62 ++++++++++++++++++++++++++-----------
 src/components/BoardForm.js |  2 --
 src/components/BoardPage.js |  3 +-
 src/components/Boards.js    | 20 ++++++------
 src/components/Post.js      | 33 ++++++++++++++++----
 src/containers/Board.js     |  6 ++--
 src/containers/Boards.js    |  5 ++-
 src/containers/WithBoard.js |  2 +-
 src/containers/WithStats.js | 44 ++++++++++++++++++++++++++
 src/index.js                | 10 +++---
 src/orbitdb/index.js        | 30 ++++++++----------
 src/reducers/boards.js      |  4 +--
 src/sagas/boards.js         | 14 ++++++---
 src/utils/ipfs.js           | 41 ++++++++++++++++++++++++
 yarn.lock                   |  4 +++
 16 files changed, 212 insertions(+), 69 deletions(-)
 create mode 100644 src/containers/WithStats.js

diff --git a/package.json b/package.json
index f5beaee..9d53487 100644
--- a/package.json
+++ b/package.json
@@ -5,6 +5,7 @@
   "private": true,
   "dependencies": {
     "ipfs": "^0.27.7",
+    "moment": "^2.20.1",
     "orbit-db": "^0.19.4",
     "orbit-db-discussion-board": "https://github.com/fazo96/orbit-db-discussion-board.git",
     "react": "^16.2.0",
diff --git a/src/components/Board.js b/src/components/Board.js
index fd6c1f3..ff5a9c0 100644
--- a/src/components/Board.js
+++ b/src/components/Board.js
@@ -1,12 +1,23 @@
 import React from 'react'
 import Post from './Post'
-import { Divider, Icon, Grid, Segment, Header, List, Button, Card } from 'semantic-ui-react'
+import { Divider, Icon, Grid, Header, List, Button, Card } from 'semantic-ui-react'
 import { Link } from 'react-router-dom'
 import { shortenAddress } from '../utils/orbitdb';
+import moment from 'moment'
 
-export default function Board({ address, posts, metadata, replicating }) {
+export default function Board({ address, posts, metadata, replicating, stats, replicationInfo, lastReplicated }) {
     const { email, website, title } = metadata || {}
-    const url = window.location.href
+    const peerCount = (stats.peers || []).length
+    const online = peerCount > 0
+    const writeable = stats.access ? (stats.access.writeable ? 'Yes' : 'No') : '?'
+    let replicationMessage = lastReplicated ? ('Last Activity at ' + moment(lastReplicated).format('H:mm')) : 'No Activity'
+    if (replicating) {
+        if (replicationInfo && replicationInfo.max !== undefined) {
+            replicationMessage = 'Progress: ' + (replicationInfo.progress || 0) + '/' + replicationInfo.max
+        } else {
+            replicationMessage = 'Initializing Transfer'
+        }
+    }
     return <Grid container divided colums={2}>
         <Grid.Column width={8}>
             <Header size='large' style={{marginTop:'.5em'}}>
@@ -18,19 +29,41 @@ export default function Board({ address, posts, metadata, replicating }) {
                 <List.Item>
                     <List.Icon name='linkify' size="large" verticalAlign="middle"/>
                     <List.Content>
-                        <List.Header>Board Address</List.Header>
+                        <List.Header>Address</List.Header>
                         <List.Content>
-                            <a href={url}>{address}</a>
+                            {address}
                         </List.Content>
                     </List.Content>
                 </List.Item>
                 <List.Item>
-                    <List.Icon name='users' size="large" verticalAlign="middle"/>
+                    <List.Icon name='disk outline' size="large" verticalAlign="middle"/>
                     <List.Content>
-                        <List.Header>Users</List.Header>
-                        <List.Content>?</List.Content>
+                        <List.Header>Size</List.Header>
+                        <List.Content>{stats.opLogLength || 0} Entries</List.Content>
                     </List.Content>
                 </List.Item>
+                <List.Item>
+                    <List.Icon name={online ? 'heart' : 'heartbeat'} size="large" verticalAlign="middle"/>
+                    <List.Content>
+                        <List.Header>{online ? 'Online' : 'Offline'}</List.Header>
+                        <List.Content>{online ? peerCount + ' Connections' : 'No Connections'}</List.Content>
+                    </List.Content>
+                </List.Item>
+                <List.Item>
+                    <List.Icon color={replicating ? 'green' : null} name='feed' size="large" verticalAlign="middle"/>
+                    <List.Content>
+                        <List.Header>{replicating ? 'Downloading' : 'Download'}</List.Header>
+                        <List.Content>{replicationMessage}</List.Content>
+                    </List.Content>
+                </List.Item>
+                <List.Item>
+                    <List.Icon name='edit' size="large" verticalAlign="middle"/>
+                    <List.Content>
+                        <List.Header>Write Access</List.Header>
+                        <List.Content>{writeable}</List.Content>
+                    </List.Content>
+                </List.Item>
+                <Divider/>
                 <List.Item>
                     <List.Icon name='file text outline' size="large" verticalAlign="middle"/>
                     <List.Content>
@@ -38,13 +71,6 @@ export default function Board({ address, posts, metadata, replicating }) {
                         <List.Content>{Object.values(posts || {}).length}</List.Content>
                     </List.Content>
                 </List.Item>
-                <List.Item>
-                    <List.Icon color={replicating ? 'green' : null} name='wifi' size="large" verticalAlign="middle"/>
-                    <List.Content>
-                        <List.Header>Replication</List.Header>
-                        <List.Content>{replicating ? 'Receiving Content...' : 'Idle'}</List.Content>
-                    </List.Content>
-                </List.Item>
                 <List.Item>
                     <List.Icon name='globe' size="large" verticalAlign="middle"/>
                     <List.Content>
@@ -61,13 +87,13 @@ export default function Board({ address, posts, metadata, replicating }) {
                 </List.Item>
             </List>
             <div className='ui three buttons basic'>
-                <Button>
+                <Button as={Link} to={'/'}>
                     <Icon name='left arrow'/> Boards
                 </Button>
-                <Button>
+                <Button disabled={!writeable}>
                     <Icon name='pencil'/> Edit
                 </Button>
-                <Button as={Link} to={shortenAddress(address)+'/p/new'}>
+                <Button disabled={!writeable} as={Link} to={shortenAddress(address)+'/p/new'}>
                     <Icon name='plus'/> New Post
                 </Button>
             </div>
diff --git a/src/components/BoardForm.js b/src/components/BoardForm.js
index 3c0d4d6..8fd4778 100644
--- a/src/components/BoardForm.js
+++ b/src/components/BoardForm.js
@@ -1,7 +1,5 @@
 import React, { Component } from 'react'
-import { Redirect } from 'react-router-dom'
 import { Container, Card, Form, Button } from 'semantic-ui-react'
-import { shortenAddress } from '../utils/orbitdb'
 
 export default class BoardForm extends Component {
     constructor(props){
diff --git a/src/components/BoardPage.js b/src/components/BoardPage.js
index 5cdb210..18518db 100644
--- a/src/components/BoardPage.js
+++ b/src/components/BoardPage.js
@@ -2,11 +2,12 @@ import React from 'react'
 import { Switch, Route } from 'react-router-dom'
 import Board from '../containers/Board'
 import PostEditor from '../containers/PostEditor'
+import WithStats from '../containers/WithStats'
 
 function BoardPage({ match, address, posts, metadata }) {
     return <Switch>
         <Route path={match.path+'p/new'} component={PostEditor} /> 
-        <Route path={match.path} component={Board} />
+        <Route path={match.path} component={WithStats(Board)} />
     </Switch>
 }
 
diff --git a/src/components/Boards.js b/src/components/Boards.js
index aba680d..33e6814 100644
--- a/src/components/Boards.js
+++ b/src/components/Boards.js
@@ -3,7 +3,7 @@ import { List, Icon, Segment, Divider, Grid, Header, Button, Card } from 'semant
 import { Link } from 'react-router-dom'
 import BoardsItem from './BoardsItem'
 
-export default function Boards({ boards, createBoard }) {
+export default function Boards({ stats, boards, createBoard }) {
     return <Grid container divided colums={2}>
         <Grid.Column width={8}>
             <Header size='large' style={{marginTop:'.5em'}}>
@@ -13,43 +13,43 @@ export default function Boards({ boards, createBoard }) {
             <Divider />
             <List relaxed>
                 <List.Item>
-                    <List.Icon name='file text outline' size="large" verticalAlign="middle"/>
+                    <List.Icon name='leaf' size="large" verticalAlign="middle"/>
                     <List.Content>
-                        <List.Header>Boards</List.Header>
-                        <List.Content>{Object.keys(boards).length}</List.Content>
+                        <List.Header>Seeding</List.Header>
+                        <List.Content>{Object.keys(boards).length} Boards</List.Content>
                     </List.Content>
                 </List.Item>
                 <List.Item>
                     <List.Icon name='wifi' size="large" verticalAlign="middle"/>
                     <List.Content>
                         <List.Header>Connected Peers</List.Header>
-                        <List.Content>?</List.Content>
+                        <List.Content>{stats.peers.length}</List.Content>
                     </List.Content>
                 </List.Item>
                 <List.Item>
                     <List.Icon name='disk outline' size="large" verticalAlign="middle"/>
                     <List.Content>
                         <List.Header>Used Space</List.Header>
-                        <List.Content>?</List.Content>
+                        <List.Content>Not Supported Yet</List.Content>
                     </List.Content>
                 </List.Item>
                 <List.Item>
-                    <List.Icon name='user' size="large" verticalAlign="middle"/>
+                    <List.Icon name='user circle' size="large" verticalAlign="middle"/>
                     <List.Content>
                         <List.Header>IPFS ID</List.Header>
-                        <List.Content>?</List.Content>
+                        <List.Content>{stats.id}</List.Content>
                     </List.Content>
                 </List.Item>
                 <List.Item>
                     <List.Icon name='key' size="large" verticalAlign="middle"/>
                     <List.Content>
                         <List.Header>OrbitDB Public Key</List.Header>
-                        <List.Content>?</List.Content>
+                        <List.Content style={{wordBreak:'break-all'}}>{stats.pubKey}</List.Content>
                     </List.Content>
                 </List.Item>
             </List>
             <div className="ui two buttons">
-                <Button as='a' href="https://github.com/fazo96/ipfs-boards">
+                <Button as='a' href="https://github.com/fazo96/ipfs-boards" target="__blank" >
                     <Icon name="github"/> GitHub
                 </Button>
                 <Button as={Link} to={'/b/new'}>
diff --git a/src/components/Post.js b/src/components/Post.js
index 596dd50..a00fca0 100644
--- a/src/components/Post.js
+++ b/src/components/Post.js
@@ -1,7 +1,7 @@
 import React from 'react'
-import { Card, Icon } from 'semantic-ui-react'
+import { List, Card } from 'semantic-ui-react'
 
-export default function Post({ title, multihash}) {
+export default function Post({ title, multihash, pubKey }) {
     return <Card fluid>
         <Card.Content>
             <Card.Header>
@@ -10,10 +10,31 @@ export default function Post({ title, multihash}) {
             <Card.Meta>Post</Card.Meta>
         </Card.Content>
         <Card.Content style={{wordBreak:'break-all'}}>
-            <Icon name="chain"/> <a href={'//ipfs.io/ipfs/'+multihash}>View</a>
-        </Card.Content>
-        <Card.Content extra>
-            <Icon name="comments"/> Comments not supported yet
+            <List>
+                <List.Item>
+                    <List.Icon name="key" verticalAlign="middle"/>
+                    <List.Content>
+                        <List.Header>Signed By</List.Header>
+                        <List.Content>{pubKey || 'Unknown'}</List.Content>
+                    </List.Content>
+                </List.Item>
+                <List.Item>
+                    <List.Icon name="comments" verticalAlign="middle"/>
+                    <List.Content>
+                        <List.Header>Comments</List.Header>
+                        <List.Content>Not Supported Yet</List.Content>
+                    </List.Content>
+                </List.Item>
+                <List.Item>
+                    <List.Icon name="chain" verticalAlign="middle"/>
+                    <List.Content>
+                        <List.Header>Content</List.Header>
+                        <List.Content>
+                            <a href={'//ipfs.io/ipfs/'+multihash}>{multihash}</a>
+                        </List.Content>
+                    </List.Content>
+                </List.Item>
+            </List>
         </Card.Content>
     </Card>
 }
\ No newline at end of file
diff --git a/src/containers/Board.js b/src/containers/Board.js
index 43fdac6..6a714c9 100644
--- a/src/containers/Board.js
+++ b/src/containers/Board.js
@@ -3,9 +3,11 @@ import { connect } from 'react-redux'
 import BoardComponent from '../components/Board'
 import { getBoardAddress } from '../utils/orbitdb'
 
-function Board({ location, match, boards }) {
+function Board({ stats, location, match, boards }) {
     const { hash, name } = match.params
-    return <BoardComponent {...boards[getBoardAddress(hash, name)]} />
+    const address = getBoardAddress(hash, name)
+    const boardStats = stats.dbs[address] || {}
+    return <BoardComponent stats={boardStats} {...boards[address]} />
 }
 
 function mapStateToProps(state){
diff --git a/src/containers/Boards.js b/src/containers/Boards.js
index 2885ddc..29cea43 100644
--- a/src/containers/Boards.js
+++ b/src/containers/Boards.js
@@ -2,9 +2,12 @@ import React from 'react'
 import { connect } from 'react-redux'
 import { push } from 'react-router-redux'
 import BoardsComponent from '../components/Boards'
+import WithStats from './WithStats'
+
+const WrappedComponent = WithStats(BoardsComponent)
 
 function Boards({ boards, createBoard }) {
-    return <BoardsComponent boards={boards} createBoard={createBoard} />
+    return <WrappedComponent boards={boards} createBoard={createBoard} />
 }
 
 function mapStateToProps(state){
diff --git a/src/containers/WithBoard.js b/src/containers/WithBoard.js
index d18cb3d..8dee0bd 100644
--- a/src/containers/WithBoard.js
+++ b/src/containers/WithBoard.js
@@ -1,5 +1,5 @@
 import React, { Component } from 'react'
-import { Loader, Dimmer } from 'semantic-ui-react'
+import { Dimmer } from 'semantic-ui-react'
 import { connect } from 'react-redux'
 import { openBoard } from '../actions/board'
 import { getBoardAddress } from '../utils/orbitdb'
diff --git a/src/containers/WithStats.js b/src/containers/WithStats.js
new file mode 100644
index 0000000..2861c4a
--- /dev/null
+++ b/src/containers/WithStats.js
@@ -0,0 +1,44 @@
+import React, { Component } from 'react'
+import { getStats } from '../utils/ipfs'
+
+export default function(WrappedComponent) {
+    return class extends Component {
+        constructor(props) {
+            super(props)
+            this.state = {
+                stats: {
+                    id: '?',
+                    peers: [],
+                    pubKey: '?',
+                    dbs: {}
+                },
+                interval: null
+            }
+        }
+
+        async refresh(loop = true) {
+            const newStats = await getStats()
+            const stats = Object.assign({}, this.state.stats, newStats)
+            this.setState({ stats }, loop ? this.refreshDelayed.bind(this) : undefined)
+        }
+
+        refreshDelayed() {
+            const timeout = setTimeout(() => {
+                this.refresh()
+            }, 2000)
+            this.setState({ timeout })
+        }
+        
+        componentDidMount() {
+            this.refresh()
+        }
+
+        componentWillUnmount() {
+            if (this.state.interval) clearInterval(this.state.interval)
+        }
+
+        render() {
+            return <WrappedComponent stats={this.state.stats} {...this.props} />
+        }
+    }
+}
\ No newline at end of file
diff --git a/src/index.js b/src/index.js
index 80dea42..e9c8884 100644
--- a/src/index.js
+++ b/src/index.js
@@ -7,6 +7,7 @@ import App from './components/App'
 import registerServiceWorker from './registerServiceWorker'
 import { Provider } from 'react-redux'
 import { ConnectedRouter } from 'react-router-redux'
+import { start } from './orbitdb'
 
 const store = configureStore();
 
@@ -23,7 +24,7 @@ render(
 
 if (module.hot) {
   module.hot.accept('./components/App', () => {
-    const NewApp = require('./components/App').default;
+    const NewApp = require('./components/App').default
     render(
       <AppContainer>
         <Provider store={store}>
@@ -33,8 +34,9 @@ if (module.hot) {
         </Provider>
       </AppContainer>,
       document.getElementById('root')
-    );
-  });
+    )
+  })
 }
 
-registerServiceWorker();
\ No newline at end of file
+registerServiceWorker()
+start(store.dispatch)
\ No newline at end of file
diff --git a/src/orbitdb/index.js b/src/orbitdb/index.js
index 6d39021..c2afaee 100644
--- a/src/orbitdb/index.js
+++ b/src/orbitdb/index.js
@@ -2,7 +2,6 @@ import IPFS from 'ipfs'
 import OrbitDB from 'orbit-db'
 import BoardStore from 'orbit-db-discussion-board'
 import multihashes from 'multihashes'
-import { getBoardIdFromAddress } from '../utils/orbitdb'
 
 export function isValidID(id) {
   try {
@@ -13,7 +12,7 @@ export function isValidID(id) {
   return false
 }
 
-export async function open(address, metadata) {
+export async function start() {
   if (!window.ipfs) {
     window.ipfs = new IPFS({
       repo: 'ipfs-v6-boards-v0',
@@ -38,23 +37,23 @@ export async function open(address, metadata) {
     OrbitDB.addDatabaseType(BoardStore.type, BoardStore)
     window.orbitDb = new OrbitDB(window.ipfs)
   }
+}
+
+export async function open(address, metadata) {
+  await start()
   const options = {
     type: BoardStore.type,
     create: true,
     write: ['*']
   }
-  try {
-    const db = await window.orbitDb.open(address, options)
-    await db.load()
-    if (metadata) {
-      await db.updateMetadata(metadata)
-    }
-    if (!window.dbs) window.dbs = {}
-    window.dbs[db.address.toString()] = db
-    return db
-  } catch (error) {
-    console.log(error)
+  const db = await window.orbitDb.open(address, options)
+  await db.load()
+  if (metadata) {
+    await db.updateMetadata(metadata)
   }
+  if (!window.dbs) window.dbs = {}
+  window.dbs[db.address.toString()] = db
+  return db
 }
 
 export function connectDb(db, dispatch) {
@@ -83,10 +82,7 @@ export function connectDb(db, dispatch) {
         progress,
         have,
         time: Date.now(),
-        replicationInfo: {
-          progress: db._replicationInfo.progress,
-          max: db._replicationInfo.max
-        }
+        replicationInfo: Object.assign({}, db._replicationInfo)
     })
   })
   db.events.on('replicate', address => {
diff --git a/src/reducers/boards.js b/src/reducers/boards.js
index 4a2d6a3..21fa4c9 100644
--- a/src/reducers/boards.js
+++ b/src/reducers/boards.js
@@ -5,7 +5,6 @@ import {
     ORBITDB_REPLICATED,
     ORBITDB_REPLICATE
 } from '../actions/actionTypes'
-import { getBoardIdFromAddress } from '../utils/orbitdb'
 
 function getInitialState() {
     return {
@@ -43,7 +42,8 @@ export default function BoardsReducer(state = getInitialState(), action) {
         case ORBITDB_REPLICATED:
             address = action.address
             return Object.assign({}, state, { boards: updateBoard(state.boards, address, {
-                replicating: false
+                replicating: false,
+                lastReplicated: action.time
             })})
         default:
             return state;
diff --git a/src/sagas/boards.js b/src/sagas/boards.js
index f581a43..92ac96e 100644
--- a/src/sagas/boards.js
+++ b/src/sagas/boards.js
@@ -2,8 +2,8 @@ import { put, call, fork, take } from 'redux-saga/effects'
 import { eventChannel } from 'redux-saga'
 import { push } from 'react-router-redux'
 import { open, connectDb } from '../orbitdb'
-import { creatingBoard, createdBoard, boardError } from '../actions/board'
-import { getBoardIdFromAddress, shortenAddress } from '../utils/orbitdb'
+import { createdBoard, boardError } from '../actions/board'
+import { shortenAddress } from '../utils/orbitdb'
 import { UPDATE_BOARD } from '../actions/actionTypes'
 
 export function* goToBoard({ board }){
@@ -35,9 +35,13 @@ export function* openBoard({ board }) {
         const dbInfo = { address }
         dbInfo.posts = db.posts
         dbInfo.metadata = db.metadata
-        const channel = yield call(createDbEventChannel, db)
-        yield fork(watchDb, channel)
-        yield put(createdBoard(Object.assign({}, board, dbInfo)))
+        try {
+            const channel = yield call(createDbEventChannel, db)
+            yield fork(watchDb, channel)
+            yield put(createdBoard(Object.assign({}, board, dbInfo)))
+        } catch (error) {
+            yield put({ type: 'ERROR', error })
+        }
     }
 }
 
diff --git a/src/utils/ipfs.js b/src/utils/ipfs.js
index 95779f3..8720286 100644
--- a/src/utils/ipfs.js
+++ b/src/utils/ipfs.js
@@ -5,4 +5,45 @@ export async function ipfsPut(content) {
     }
     const response = await window.ipfs.files.add(obj)
     return response[0].hash
+}
+
+export async function readText(multihash) {
+    const buffer = await window.ipfs.object.get(multihash)
+    return buffer.toString('utf-8')
+}
+
+export async function getStats() {
+    const ipfs = window.ipfs;
+    const orbitDb = window.orbitDb
+    const dbs = {}
+    const stats = {}
+    if (ipfs) {
+        const peers = await ipfs.swarm.peers()
+        const id = await ipfs.id()
+        stats.peers = peers.map(p => p.peer.id._idB58String)
+        stats.id = id.id
+    }
+    if (orbitDb) {
+        stats.pubKey = await orbitDb.key.getPublic('hex')
+        Object.values(window.dbs || {}).forEach(db => {
+            let writeable = db.access.write.indexOf('*') >= 0 || db.access.write.indexOf(stats.pubKey) >= 0
+            const dbInfo = {
+                opLogLength: db._oplog.length,
+                access: {
+                    admin: db.access.admin,
+                    read: db.access.read,
+                    write: db.access.write,
+                    writeable
+                },
+                peers: []
+            }
+            const subscription = orbitDb._pubsub._subscriptions
+            if (subscription && subscription.room) {
+                dbInfo.peers = subscription.room._peers
+            }
+            dbs[db.address] = dbInfo
+        }) 
+    }
+    stats.dbs = dbs
+    return stats
 }
\ No newline at end of file
diff --git a/yarn.lock b/yarn.lock
index 438cba1..c8ad03a 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -5862,6 +5862,10 @@ mkdirp@0.5.1, mkdirp@0.5.x, "mkdirp@>=0.5 0", mkdirp@^0.5.1, mkdirp@~0.5.0, mkdi
   dependencies:
     minimist "0.0.8"
 
+moment@^2.20.1:
+  version "2.20.1"
+  resolved "https://registry.yarnpkg.com/moment/-/moment-2.20.1.tgz#d6eb1a46cbcc14a2b2f9434112c1ff8907f313fd"
+
 moving-average@^1.0.0:
   version "1.0.0"
   resolved "https://registry.yarnpkg.com/moving-average/-/moving-average-1.0.0.tgz#b1247ba8dd2d7927c619f1eac8036cf933d65adc"