From d3aff4a11056dc2480eca4aab759fd1866f4bc19 Mon Sep 17 00:00:00 2001
From: "Daniel J. Summers" <daniel.summers.2415@gmail.com>
Date: Fri, 17 Aug 2018 20:59:57 -0500
Subject: [PATCH] Add/Edit request works

No longer a modal (#20)
---
 src/api/MyPrayerJournal.Api/Handlers.fs       |   2 +-
 src/app/src/App.vue                           |  12 ++
 src/app/src/api/index.js                      |   2 +-
 src/app/src/components/Journal.vue            |  11 +-
 .../src/components/request/EditRequest.vue    | 148 ++++++++++++------
 .../src/components/request/RequestCard.vue    |  30 ++--
 src/app/src/router/index.js                   |   7 +
 src/app/src/store/index.js                    |   8 +-
 8 files changed, 135 insertions(+), 85 deletions(-)

diff --git a/src/api/MyPrayerJournal.Api/Handlers.fs b/src/api/MyPrayerJournal.Api/Handlers.fs
index 7df9836..5073d76 100644
--- a/src/api/MyPrayerJournal.Api/Handlers.fs
+++ b/src/api/MyPrayerJournal.Api/Handlers.fs
@@ -26,7 +26,7 @@ module Error =
   /// Handle 404s from the API, sending known URL paths to the Vue app so that they can be handled there
   let notFound : HttpHandler =
     fun next ctx ->
-      [ "/answered"; "/journal"; "/snoozed"; "/user" ]
+      [ "/answered"; "/journal"; "/legal"; "/request"; "/snoozed"; "/user" ]
       |> List.filter ctx.Request.Path.Value.StartsWith
       |> List.length
       |> function
diff --git a/src/app/src/App.vue b/src/app/src/App.vue
index f68c23d..be30745 100644
--- a/src/app/src/App.vue
+++ b/src/app/src/App.vue
@@ -61,8 +61,17 @@ a[role="button"] {
   border-radius: .5rem;
   background-color: whitesmoke;
 }
+a[role="button"] {
+  padding: .25rem;
+}
+button.primary,
+a[role="button"].primary {
+  background-color: white;
+  border-width: 3px;
+}
 button:hover,
 a[role="button"]:hover {
+  cursor: pointer;
   background-color: #050;
   color: white;
   text-decoration: none;
@@ -132,4 +141,7 @@ a:hover {
   margin-left: 1rem;
   margin-right: 1rem;
 }
+.material-icons {
+  vertical-align: middle;
+}
 </style>
diff --git a/src/app/src/api/index.js b/src/app/src/api/index.js
index 5d24c4d..5720496 100644
--- a/src/app/src/api/index.js
+++ b/src/app/src/api/index.js
@@ -31,7 +31,7 @@ export default {
    * Add a new prayer request
    * @param {string} requestText The text of the request to be added
    */
-  addRequest: requestText => http.post('request', { requestText }),
+  addRequest: requestText => http.post('request', { requestText, recurType: 'immediate' }),
 
   /**
    * Get all answered requests, along with the text they had when it was answered
diff --git a/src/app/src/components/Journal.vue b/src/app/src/components/Journal.vue
index 6cee922..d2747be 100644
--- a/src/app/src/components/Journal.vue
+++ b/src/app/src/components/Journal.vue
@@ -3,7 +3,10 @@ article.mpj-main-content-wide(role='main')
   page-title(:title='title')
   p(v-if='isLoadingJournal') Loading your prayer journal...
   template(v-else)
-    new-request
+    router-link(:to="{ name: 'EditRequest', params: { id: 'new' } }"
+                role='button')
+      md-icon(icon='add_box')
+      | &nbsp; Add a New Request
     br
     .mpj-journal(v-if='journal.length > 0')
       request-card(v-for='request in journal'
@@ -13,8 +16,6 @@ article.mpj-main-content-wide(role='main')
                    :toast='toast')
     p.text-center(v-else): em.
       No requests found; click the &ldquo;Add a New Request&rdquo; button to add one
-    edit-request(:events='eventBus'
-                 :toast='toast')
     notes-edit(:events='eventBus'
                :toast='toast')
     snooze-request(:events='eventBus'
@@ -27,8 +28,6 @@ article.mpj-main-content-wide(role='main')
 import Vue from 'vue'
 import { mapState } from 'vuex'
 
-import EditRequest from './request/EditRequest'
-import NewRequest from './request/NewRequest'
 import NotesEdit from './request/NotesEdit'
 import RequestCard from './request/RequestCard'
 import SnoozeRequest from './request/SnoozeRequest'
@@ -38,8 +37,6 @@ import actions from '@/store/action-types'
 export default {
   name: 'journal',
   components: {
-    EditRequest,
-    NewRequest,
     NotesEdit,
     RequestCard,
     SnoozeRequest
diff --git a/src/app/src/components/request/EditRequest.vue b/src/app/src/components/request/EditRequest.vue
index 77e9544..235cd12 100644
--- a/src/app/src/components/request/EditRequest.vue
+++ b/src/app/src/components/request/EditRequest.vue
@@ -1,46 +1,62 @@
 <template lang="pug">
-b-modal(v-model='editVisible'
-        header-bg-variant='mpj'
-        header-text-variant='light'
-        size='lg'
-        title='Edit Prayer Request'
-        @edit='openDialog()'
-        @shows='focusRequestText')
-  b-form
-    b-form-group(label='Prayer Request'
-                 label-for='request_text')
-      b-textarea#request_text(ref='toFocus'
-                              v-model='form.requestText'
-                              :rows='10'
-                              @blur='trimText()')
-    b-form-group(label='Also Mark As')
-      b-radio-group(v-model='form.status'
-                    buttons)
-        b-radio(value='Updated') Updated
-        b-radio(value='Prayed') Prayed
-        b-radio(value='Answered') Answered
-  div.w-100.text-right(slot='modal-footer')
-    b-btn(variant='primary'
-          @click='saveRequest()') Save
+article.mpj-main-content(role='main')
+  page-title(:title='title')
+  form.mpj-edit-request-form
+    label(for='request_text') Prayer Request
+    br
+    textarea#request_text(v-model='form.requestText'
+                          :rows='10'
+                          @blur='trimText()'
+                          autofocus)
+    br
+    template(v-if='!isNew')
+      label Also Mark As
+      br
+      input(type='radio'
+            id='status_updated'
+            value='Updated'
+            v-model='form.status')
+      label(for='status_updated')= ' Updated'
+      input(type='radio'
+            id='status_prayed'
+            value='Prayed'
+            v-model='form.status')
+      label(for='status_prayed')= ' Prayed'
+      input(type='radio'
+            id='status_answered'
+            value='Answered'
+            v-model='form.status')
+      label(for='status_answered')= ' Answered'
+    br(v-else)
+  div.mpj-edit-request-form.mpj-text-right
+    button(@click.stop='saveRequest()').primary
+      md-icon(icon='save')
+      = ' Save'
     | &nbsp; &nbsp;
-    b-btn(variant='outline-secondary'
-          @click='closeDialog()') Cancel
+    button(@click.stop='goBack()')
+      md-icon(icon='arrow_back')
+      = ' Cancel'
 </template>
 
 <script>
 'use strict'
 
+import { mapState } from 'vuex'
+
 import actions from '@/store/action-types'
 
 export default {
   name: 'edit-request',
   props: {
-    toast: { required: true },
-    events: { required: true }
+    id: {
+      type: String,
+      required: true
+    }
   },
   data () {
     return {
-      editVisible: false,
+      title: 'Edit Prayer Request',
+      isNew: false,
       form: {
         requestId: '',
         requestText: '',
@@ -48,42 +64,70 @@ export default {
       }
     }
   },
-  created () {
-    this.events.$on('edit', this.openDialog)
+  computed: {
+    toast () {
+      return this.$parent.$refs.toast
+    },
+    ...mapState(['journal'])
   },
-  methods: {
-    closeDialog () {
+  async mounted () {
+    if (this.id === 'new') {
+      this.title = 'Add Prayer Request'
+      this.isNew = true
       this.form.requestId = ''
       this.form.requestText = ''
+      this.form.status = 'Created'
+    } else {
+      this.title = 'Edit Prayer Request'
+      this.isNew = false
+      if (this.journal.length === 0) {
+        await this.$store.dispatch(actions.LOAD_JOURNAL, this.$Progress)
+      }
+      const req = this.journal.filter(r => r.requestId === this.id)[0]
+      this.form.requestId = this.id
+      this.form.requestText = req.text
       this.form.status = 'Updated'
-      this.editVisible = false
-    },
-    focusRequestText (e) {
-      this.$refs.toFocus.focus()
-    },
-    openDialog (request) {
-      this.form.requestId = request.requestId
-      this.form.requestText = request.text
-      this.editVisible = true
-      this.focusRequestText(null)
+    }
+  },
+  methods: {
+    goBack () {
+      this.$router.go(-1)
     },
     trimText () {
       this.form.requestText = this.form.requestText.trim()
     },
     async saveRequest () {
-      await this.$store.dispatch(actions.UPDATE_REQUEST, {
-        progress: this.$Progress,
-        requestId: this.form.requestId,
-        updateText: this.form.requestText,
-        status: this.form.status
-      })
-      if (this.form.status === 'Answered') {
-        this.toast.showToast('Request updated and removed from active journal', { theme: 'success' })
+      if (this.isNew) {
+        await this.$store.dispatch(actions.ADD_REQUEST, {
+          progress: this.$Progress,
+          requestText: this.form.requestText
+        })
+        this.toast.showToast('New prayer request added', { theme: 'success' })
       } else {
-        this.toast.showToast('Request updated', { theme: 'success' })
+        await this.$store.dispatch(actions.UPDATE_REQUEST, {
+          progress: this.$Progress,
+          requestId: this.form.requestId,
+          updateText: this.form.requestText,
+          status: this.form.status
+        })
+        if (this.form.status === 'Answered') {
+          this.toast.showToast('Request updated and removed from active journal', { theme: 'success' })
+        } else {
+          this.toast.showToast('Request updated', { theme: 'success' })
+        }
       }
-      this.closeDialog()
+      this.goBack()
     }
   }
 }
 </script>
+
+<style>
+.mpj-edit-request-form {
+  max-width: 40rem;
+  margin: auto;
+}
+#request_text {
+  width: 100%;
+}
+</style>
diff --git a/src/app/src/components/request/RequestCard.vue b/src/app/src/components/request/RequestCard.vue
index 850ab20..bb226b9 100644
--- a/src/app/src/components/request/RequestCard.vue
+++ b/src/app/src/components/request/RequestCard.vue
@@ -1,10 +1,10 @@
 <template lang="pug">
 .mpj-request-card
-  header.mpj-card-header.mpj-bg(role='toolbar').
-    #[button.pray(@click='markPrayed()' title='Pray' size='sm'): md-icon(icon='done')]
-    #[button(@click.stop='showEdit()' title='Edit' size='sm'): md-icon(icon='edit')]
-    #[button(@click.stop='showNotes()' title='Add Notes' size='sm'): md-icon(icon='comment')]
-    #[button(@click.stop='snooze()' title='Snooze Request' size='sm'): md-icon(icon='schedule')]
+  header.mpj-card-header(role='toolbar').
+    #[button.primary(@click='markPrayed()' title='Pray'): md-icon(icon='done')]
+    #[button(@click.stop='showEdit()' title='Edit'): md-icon(icon='edit')]
+    #[button(@click.stop='showNotes()' title='Add Notes'): md-icon(icon='comment')]
+    #[button(@click.stop='snooze()' title='Snooze Request'): md-icon(icon='schedule')]
   div
     p.card-text.mpj-request-text
       | {{ request.text }}
@@ -42,10 +42,7 @@ export default {
       this.toast.showToast('Request marked as prayed', { theme: 'success' })
     },
     showEdit () {
-      this.events.$emit('edit', this.request)
-    },
-    showFull () {
-      this.events.$emit('full', this.request.requestId)
+      this.$router.push({ name: 'EditRequest', params: { id: this.request.requestId } })
     },
     showNotes () {
       this.events.$emit('notes', this.request)
@@ -73,22 +70,15 @@ export default {
   display: flex;
   flex-flow: row;
   justify-content: center;
+  background-image: -webkit-gradient(linear, left top, left bottom, from(lightgray), to(whitesmoke));
+  background-image: -webkit-linear-gradient(top, lightgray, whitesmoke);
+  background-image: -moz-linear-gradient(top, lightgray, whitesmoke);
+  background-image: linear-gradient(to bottom, lightgray, whitesmoke);
 }
 .mpj-card-header button {
-  background-color: rgba(255, 255, 255, .75);
-  border-radius: .25rem;
   margin: .25rem;
-  border: solid #050 1px;
   font-size: .8rem;
 }
-.mpj-card-header button:hover {
-  cursor: pointer;
-  background-color: white;
-  color: #050;
-}
-.mpj-card-header button.pray {
-  background-color: white;
-}
 .mpj-request-card .card-text {
   margin-left: 1rem;
   margin-right: 1rem;
diff --git a/src/app/src/router/index.js b/src/app/src/router/index.js
index 1e75a2f..ea2f9d5 100644
--- a/src/app/src/router/index.js
+++ b/src/app/src/router/index.js
@@ -3,6 +3,7 @@ import Router from 'vue-router'
 
 import Answered from '@/components/Answered'
 import AnsweredDetail from '@/components/AnsweredDetail'
+import EditRequest from '@/components/request/EditRequest'
 import Home from '@/components/Home'
 import Journal from '@/components/Journal'
 import LogOn from '@/components/user/LogOn'
@@ -46,6 +47,12 @@ export default new Router({
       name: 'TermsOfService',
       component: TermsOfService
     },
+    {
+      path: '/request/:id/edit',
+      name: 'EditRequest',
+      component: EditRequest,
+      props: true
+    },
     {
       path: '/snoozed',
       name: 'Snoozed',
diff --git a/src/app/src/store/index.js b/src/app/src/store/index.js
index 4f8c939..74ee607 100644
--- a/src/app/src/store/index.js
+++ b/src/app/src/store/index.js
@@ -9,7 +9,7 @@ import actions from './action-types'
 
 Vue.use(Vuex)
 
-this.auth0 = new AuthService()
+const auth0 = new AuthService()
 
 const logError = function (error) {
   if (error.response) {
@@ -34,11 +34,11 @@ export default new Vuex.Store({
   state: {
     user: JSON.parse(localStorage.getItem('user_profile') || '{}'),
     isAuthenticated: (() => {
-      this.auth0.scheduleRenewal()
-      if (this.auth0.isAuthenticated()) {
+      auth0.scheduleRenewal()
+      if (auth0.isAuthenticated()) {
         api.setBearer(localStorage.getItem('id_token'))
       }
-      return this.auth0.isAuthenticated()
+      return auth0.isAuthenticated()
     })(),
     journal: {},
     isLoadingJournal: false