From 1144d704817e7f006e0147eec76034e20b2119b7 Mon Sep 17 00:00:00 2001
From: Jaakko Lappalainen <jkk.lapp@gmail.com>
Date: Wed, 8 Jun 2022 16:26:31 +0100
Subject: [PATCH] feat(Posts): create comments for posts

---
 frontend/src/components/Dashboard/Posts.vue |  4 ++--
 frontend/src/store/actions/index.js         | 12 ++++++++++--
 frontend/src/store/index.js                 |  1 +
 frontend/src/store/mutations.js             |  9 ++++++---
 frontend/src/store/mutations.spec.js        |  2 +-
 5 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/frontend/src/components/Dashboard/Posts.vue b/frontend/src/components/Dashboard/Posts.vue
index ac9de12..feb0dfc 100644
--- a/frontend/src/components/Dashboard/Posts.vue
+++ b/frontend/src/components/Dashboard/Posts.vue
@@ -33,7 +33,6 @@
         <div class="flex flex-wrap place-items-end">
           <div class="w-100 mr-auto"></div>
           <a
-            v-show="p.comments > 0"
             href="#"
             class="text-gray-500 dark:text-gray-400"
             @click.prevent="() => fetchPosts(p.id)"
@@ -118,7 +117,8 @@ export default {
       return moment(date).fromNow();
     },
     fetchPosts(parentId) {
-      this.$store.dispatch('fetchPosts', { ...this.$store.state, parentId });
+      this.$store.dispatch('setParentId', parentId);
+      this.$store.dispatch('fetchPosts', this.$store.state);
     },
     reset() {
       this.$store.dispatch('resetPostsPagination');
diff --git a/frontend/src/store/actions/index.js b/frontend/src/store/actions/index.js
index da3819c..9e8280a 100644
--- a/frontend/src/store/actions/index.js
+++ b/frontend/src/store/actions/index.js
@@ -5,6 +5,9 @@ export default {
     commit('SET_USER', user);
     commit('SET_LOGGED_IN', user.email && user.displayName);
   },
+  setParentId({ commit }, parentId) {
+    commit('SET_PARENT_ID', parentId);
+  },
   async fetchPosts({ commit }, { startAfter, limit, parentId }) {
     commit('IS_LOADING_POSTS', true);
     const { data } = await apiRequest('GET', '/posts', {
@@ -13,7 +16,7 @@ export default {
       parentId,
     });
     const { results, nextPageToken, remainingMessages } = data;
-    commit('SET_POSTS', { results, parentId });
+    commit('SET_POSTS', results);
     commit('SET_START_AFTER', nextPageToken);
     commit('SET_REMAINING_MESSAGES', remainingMessages || 0);
     commit('IS_LOADING_POSTS', false);
@@ -23,15 +26,20 @@ export default {
   },
   resetPostsPagination({ commit }) {
     commit('SET_START_AFTER', null);
+    commit('SET_PARENT_ID', null);
   },
   async postMessage({ commit, state }, message) {
     commit('IS_CREATING_POST', true);
     commit('PUSH_MESSAGE', {
       message,
       date: new Date().getTime(),
+      parentId: state.parentId,
     });
     try {
-      const { data } = await apiRequest('POST', '/posts', null, { message });
+      const { data } = await apiRequest('POST', '/posts', null, {
+        message,
+        parentId: state.parentId || undefined,
+      });
       commit('POP_MESSAGE');
       commit('PUSH_MESSAGE', data);
       commit('SET_REMAINING_MESSAGES', state.remainingMessages - 1);
diff --git a/frontend/src/store/index.js b/frontend/src/store/index.js
index f0b5e69..0cc3679 100644
--- a/frontend/src/store/index.js
+++ b/frontend/src/store/index.js
@@ -20,6 +20,7 @@ const store = createStore({
     userNameExists: false,
     isCheckingEmail: false,
     isCheckingName: false,
+    parentId: null,
   },
   getters,
   mutations,
diff --git a/frontend/src/store/mutations.js b/frontend/src/store/mutations.js
index e034126..1350eb3 100644
--- a/frontend/src/store/mutations.js
+++ b/frontend/src/store/mutations.js
@@ -5,9 +5,12 @@ export default {
   SET_LOGGED_IN(state, data) {
     state.loggedIn = data;
   },
-  SET_POSTS(state, { results, parentId }) {
-    if (parentId) {
-      const parent = state.posts.find((post) => post.id === parentId);
+  SET_PARENT_ID(state, data) {
+    state.parentId = data;
+  },
+  SET_POSTS(state, results) {
+    if (state.parentId) {
+      const parent = state.posts.find((post) => post.id === state.parentId);
       state.posts = [parent, ...results];
     } else {
       state.posts = results;
diff --git a/frontend/src/store/mutations.spec.js b/frontend/src/store/mutations.spec.js
index 1700963..1bec93b 100644
--- a/frontend/src/store/mutations.spec.js
+++ b/frontend/src/store/mutations.spec.js
@@ -26,7 +26,7 @@ describe('mutations', () => {
       // mock state
       const state = { posts: [] };
       // apply mutation
-      SET_POSTS(state, { results: postsFixture });
+      SET_POSTS(state, postsFixture);
       // assert result
       expect(state.posts).toEqual(postsFixture);
     });