Update statistics to round average words

Dantaro · Jun 16, 2020 · e2467dd · e2467dd
1 parent b844485
commit e2467dd
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 8 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "one-nice-thing",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "private": true,
   "dependencies": {
     "@fortawesome/fontawesome-svg-core": "^1.2.28",

diff --git a/src/appshell/AppShell.js b/src/appshell/AppShell.js
@@ -90,7 +90,7 @@ export default ({ children }) => {
                     </ListItem>
                 </Link>
                 <ListItem>
-                    <Typography variant="subtitle2">v1.1.0</Typography>
+                    <Typography variant="subtitle2">v1.1.1</Typography>
                 </ListItem>
             </List>
         </div>

diff --git a/src/page/statistics/Statistics.js b/src/page/statistics/Statistics.js
@@ -4,7 +4,7 @@ import Paper from '@material-ui/core/Paper'
 import { useSelector } from 'react-redux'
 import {
     findMostConsecutiveDays,
-    findAverageWordCount,
+    findAverageWordCountRounded,
     findLongestNote,
 } from 'util/notelist'
 import { makeStyles } from '@material-ui/core/styles'
@@ -25,7 +25,7 @@ export const Statistics = () => {
     const classes = useStyles()
     const noteList = useSelector((state) => state.noteList)
     const bestStreak = findMostConsecutiveDays(noteList)
-    const averageWordCount = findAverageWordCount(noteList)
+    const averageWordCount = findAverageWordCountRounded(noteList)
     const longestNote = findLongestNote(noteList)
     return (
         <div>
@@ -48,15 +48,17 @@ export const Statistics = () => {
                     {`${bestStreak} ${bestStreak === 1 ? 'day' : 'days'}`}
                 </Typography>
                 <Typography variant="body1" gutterBottom>
-                    Average word count:{' '}
-                    {`${averageWordCount} ${
-                        averageWordCount === 1 ? 'word' : 'words'
-                    }`}
+                    Average word count: {' '}
+                    {`${averageWordCount} ${averageWordCount === 1 ? 'word' : 'words'}`}
                 </Typography>
                 <Typography variant="body1" gutterBottom>
                     Longest Note:{' '}
                     {`${longestNote} ${longestNote === 1 ? 'word' : 'words'}`}
                 </Typography>
+                <br/>
+                <Typography variant="body2">
+                    Have an idea for other statistics? <a href="mailto:[email protected]">Shoot over an email</a> with your suggestions!
+                </Typography>
             </Paper>
         </div>
     )

diff --git a/src/util/notelist/NoteListUtil.js b/src/util/notelist/NoteListUtil.js
@@ -71,6 +71,9 @@ export const findAverageWordCount = (noteList) => {
     )
 }
 
+export const findAverageWordCountRounded = (noteList) => 
+    Math.round(findAverageWordCount(noteList))
+
 export const findLongestNote = (noteList) => {
     if (!noteList || noteList.length === 0) {
         return 0

diff --git a/src/util/notelist/index.js b/src/util/notelist/index.js
@@ -2,12 +2,14 @@ import {
     findMostConsecutiveDays,
     findCurrentStreak,
     findAverageWordCount,
+    findAverageWordCountRounded,
     findLongestNote,
 } from './NoteListUtil'
 
 export {
     findMostConsecutiveDays,
     findCurrentStreak,
     findAverageWordCount,
+    findAverageWordCountRounded,
     findLongestNote,
 }