Update NES hook

Author: Kevnz

readme.md

@@ -297,7 +297,7 @@ import React from 'react'
 import { useNes } from '@brightleaf/react-hooks'
 export default () => {
 
-  const { message, error, connecting, connected } = useNes(
+  const { messages, error, connecting, connected } = useNes(
     'wss://kev-pi.herokuapp.com'
   )
 
@@ -307,11 +307,11 @@ export default () => {
   const connectMessage = connecting ? <div>Connecting</div> : <div>Not Connecting</div>
   const connectedMessage = connected ? <div>Connected</div> : <div>Not Connected</div>
 
-  const messages = message.map((m, index) => <div key={`key-${index}`}>{m}</div>)
+  const messageList = messages.map((m, index) => <div key={`key-${index}`}>{m}</div>)
   return (
     <div className="App">
       <h1>Messages from Server</h1>
-      <div>{messages}</div>
+      <div>{messageList}</div>
       <div>
         <b>Status Messages</b>
         {connectMessage}

src/use-nes.js

@@ -2,6 +2,7 @@ import { useEffect, useReducer } from 'react'
 import Nes from '@hapi/nes/lib/client'
 
 const reducer = (state, action, ...other) => {
+  const messages = [].concat(state.messages)
   switch (action.type) {
     case 'connecting':
       return { ...state, connecting: true, connected: false }
@@ -10,12 +11,10 @@ const reducer = (state, action, ...other) => {
     case 'disconnected':
       return { ...state, connecting: false, connected: false }
     case 'message':
-      const message = [].concat(state.message)
-      message.push(action.payload.data)
-
+      messages.push(action.payload.data)
       return {
         ...state,
-        message,
+        messages,
         error: null,
         connecting: false,
       }
@@ -29,9 +28,9 @@ const reducer = (state, action, ...other) => {
       return state
   }
 }
-const useNes = (url = 'ws://localhost:4567', subscribe = true) => {
+const useNes = (url = 'ws://localhost:4567', subscribe) => {
   const [state, dispatch] = useReducer(reducer, {
-    message: [],
+    messages: [],
     error: null,
     connecting: true,
     connected: false,
@@ -41,8 +40,8 @@ const useNes = (url = 'ws://localhost:4567', subscribe = true) => {
   useEffect(() => {
     const connectClient = async () => {
       dispatch({ type: 'connecting', payload: {} })
-      return new Promise(async (resolve, reject) => {
-        client.onConnect = update => {
+      return new Promise(async resolve => {
+        client.onConnect = () => {
           dispatch({ type: 'connected' })
           return resolve()
         }
@@ -58,7 +57,7 @@ const useNes = (url = 'ws://localhost:4567', subscribe = true) => {
         }
 
         if (subscribe) {
-          client.subscribe(url, update => {
+          client.subscribe(subscribe, update => {
             dispatch({ type: 'message', payload: { data: update } })
             return resolve()
           })

working/hooks/nes.js

@@ -3,7 +3,7 @@ import { useNes, useTitle } from '@brightleaf/react-hooks'
 export default () => {
   useTitle('useNes example from @brightleaf/react-hooks')
 
-  const { message, error, connecting, connected, client } = useNes(
+  const { messages, error, connecting, connected, client } = useNes(
     'wss://kev-pi.herokuapp.com'
   )
 
@@ -24,7 +24,7 @@ export default () => {
   ) : (
     <div className="alert alert-danger">Not Connected</div>
   )
-  const messages = message.map((m, index) => (
+  const messageList = messages.map((m, index) => (
     <div
       className={`mar-0 pad-b-xs ${
         index % 2 ? 'text-secondary' : 'text-primary'
@@ -37,7 +37,7 @@ export default () => {
   return (
     <div className="App content">
       <h2>Messages from Server</h2>
-      <div>{messages}</div>
+      <div>{messageList}</div>
       <div className="pad-t-md mar-t-md">
         <div
           className="button"