From 67915246ba0e1720f7778a6fbee932063334e38f Mon Sep 17 00:00:00 2001
From: Hubert Van De Walle <hubv@protonmail.com>
Date: Thu, 25 Jun 2020 15:41:03 +0200
Subject: [PATCH] Update..

---
 api/src/validation/NoteValidation.kt | 16 ++++++++++++++++
 frontend/components/TagsGroup.vue    |  7 ++++++-
 frontend/nuxt.config.js              | 14 +-------------
 frontend/pages/create.vue            | 11 +++--------
 frontend/pages/notes/_uuid.vue       |  7 +++++--
 frontend/pages/notes/index.vue       | 17 ++++++++++++-----
 frontend/store/notes.js              | 22 +++++++++++++++++-----
 7 files changed, 60 insertions(+), 34 deletions(-)
 create mode 100644 api/src/validation/NoteValidation.kt

diff --git a/api/src/validation/NoteValidation.kt b/api/src/validation/NoteValidation.kt
new file mode 100644
index 0000000..0c04dc6
--- /dev/null
+++ b/api/src/validation/NoteValidation.kt
@@ -0,0 +1,16 @@
+package be.vandewalleh.validation
+
+import am.ik.yavi.builder.ValidatorBuilder
+import am.ik.yavi.builder.konstraint
+import am.ik.yavi.core.Validator
+import be.vandewalleh.entities.Note
+import be.vandewalleh.entities.User
+
+val noteValidator: Validator<Note> = ValidatorBuilder.of<Note>()
+    .konstraint(Note::title) {
+        notNull().notBlank().lessThanOrEqual(50)
+    }
+    .konstraint(Note::tags) {
+        this.lessThanOrEqual(10)
+    }
+    .build()
diff --git a/frontend/components/TagsGroup.vue b/frontend/components/TagsGroup.vue
index b9e8754..e933e76 100644
--- a/frontend/components/TagsGroup.vue
+++ b/frontend/components/TagsGroup.vue
@@ -1,6 +1,11 @@
 <template>
     <div class="tags">
-        <v-chip v-for="tag in tags" :key="tag" active color="secondary">
+        <v-chip
+            v-for="(tag, index) in tags"
+            :key="index"
+            active
+            color="secondary"
+        >
             {{ tag }}
         </v-chip>
     </div>
diff --git a/frontend/nuxt.config.js b/frontend/nuxt.config.js
index 7ed9889..26216f3 100644
--- a/frontend/nuxt.config.js
+++ b/frontend/nuxt.config.js
@@ -72,19 +72,7 @@ export default {
      ** Axios module configuration
      ** See https://axios.nuxtjs.org/options
      */
-    axios: {
-        headers: {
-            common: {
-                Accept: 'application/json',
-            },
-            delete: {},
-            get: {},
-            head: {},
-            post: {},
-            put: {},
-            patch: {},
-        },
-    },
+    axios: {},
 
     env: {
         API_HOST: env.API_HOST,
diff --git a/frontend/pages/create.vue b/frontend/pages/create.vue
index cd3b18a..c575103 100644
--- a/frontend/pages/create.vue
+++ b/frontend/pages/create.vue
@@ -21,6 +21,8 @@
                         auto-grow
                     ></v-textarea>
                     <client-only v-else>
+                        <!-- html is sanitized -->
+                        <!-- eslint-disable-next-line -->
                         <div class="html" v-html="html" />
                     </client-only>
                 </v-card-text>
@@ -51,12 +53,7 @@ export default {
         },
     },
     mounted() {
-        this.$axios
-            .get('/tags')
-            .then((e) => (this.possibleTags = e.data))
-            .catch((e) => {
-                console.log(e)
-            })
+        this.$axios.get('/tags').then((e) => (this.possibleTags = e.data))
     },
     methods: {
         createNote() {
@@ -71,8 +68,6 @@ export default {
                 .catch((e) => {
                     if (e.response && e.response.status === 409) {
                         this.conflict = true
-                    } else {
-                        console.log('??')
                     }
                 })
         },
diff --git a/frontend/pages/notes/_uuid.vue b/frontend/pages/notes/_uuid.vue
index bbf82f6..e1a2f45 100644
--- a/frontend/pages/notes/_uuid.vue
+++ b/frontend/pages/notes/_uuid.vue
@@ -6,6 +6,8 @@
             <TagsGroup :tags="note.tags" />
         </v-card-title>
         <v-card-text>
+            <!-- html is sanitized -->
+            <!-- eslint-disable-next-line -->
             <div v-html="renderedMarkdown"></div>
         </v-card-text>
     </v-card>
@@ -23,8 +25,9 @@ export default {
     }),
     computed: {
         renderedMarkdown() {
-            if (!this.note.chapters) return ''
-            return renderMarkdown(this.note.chapters[0].content).contents
+            return !this.note.content
+                ? ''
+                : renderMarkdown(this.note.content).contents
         },
     },
     mounted() {
diff --git a/frontend/pages/notes/index.vue b/frontend/pages/notes/index.vue
index df5f0fd..9e6804e 100644
--- a/frontend/pages/notes/index.vue
+++ b/frontend/pages/notes/index.vue
@@ -1,13 +1,16 @@
 <template>
     <div>
         <v-progress-circular
-            v-if="loading && notes.length === 0"
+            v-if="!isInitialized"
             :size="100"
             :width="7"
             color="purple"
             indeterminate
         ></v-progress-circular>
-        <v-row>
+        <v-card v-if="isEmpty">
+            <v-card-text>No notes yet :/</v-card-text>
+        </v-card>
+        <v-row v-else>
             <v-col
                 v-for="note in notes"
                 :key="note.uuid"
@@ -28,7 +31,7 @@
 </template>
 
 <script>
-import { mapState } from 'vuex'
+import { mapState, mapGetters, mapActions } from 'vuex'
 
 export default {
     name: 'Notes',
@@ -36,10 +39,14 @@ export default {
         loading: true,
     }),
     computed: {
-        ...mapState('notes', { notes: (state) => state.notes }),
+        ...mapState('notes', ['notes', 'isInitialized']),
+        ...mapGetters('notes', ['isEmpty']),
     },
     mounted() {
-        this.$store.dispatch('notes/load').then(() => (this.loading = false))
+        if (!this.initialized) this.load()
+    },
+    methods: {
+        ...mapActions('notes', ['load']),
     },
     head: () => ({
         title: 'My notes',
diff --git a/frontend/store/notes.js b/frontend/store/notes.js
index aa23740..69cffbc 100644
--- a/frontend/store/notes.js
+++ b/frontend/store/notes.js
@@ -1,8 +1,12 @@
 export const state = () => ({
     notes: [],
+    isInitialized: false,
 })
 
 export const mutations = {
+    setInitialized(state) {
+        state.isInitialized = true
+    },
     set(state, notes) {
         state.notes = notes
     },
@@ -15,21 +19,25 @@ export const mutations = {
     // used when logging out
     clear(state) {
         state.notes = []
+        state.initialized = false
     },
 }
 
 export const actions = {
     async load({ commit }) {
         await new Promise((resolve) => setTimeout(resolve, 600))
-        this.$axios.get('/notes').then(({ data }) => commit('set', data))
+        this.$axios.get('/notes').then(({ data }) => {
+            commit('set', data)
+            commit('setInitialized')
+        })
     },
-    // FIXME: make the api send the new date back
     create({ commit }, note) {
         this.$axios.post('/notes', note).then(({ data }) =>
             commit('add', {
-                updatedAt: new Date().toISOString(),
                 uuid: data.uuid,
-                ...note,
+                title: data.title,
+                updatedAt: data.updatedAt,
+                tags: data.tags,
             })
         )
     },
@@ -40,4 +48,8 @@ export const actions = {
     },
 }
 
-export const getters = {}
+export const getters = {
+    isEmpty(state) {
+        return state.isInitialized && state.notes.length === 0
+    },
+}