From c5a56626ecf259017522967ecc4c007ca0c91dbf Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 10:41:44 -0600
Subject: [PATCH 01/63] showing add to course dialog

---
 src/components/Dialogs/AddStudentDialog.vue   | 74 +++++++++++++++++++
 .../Training/TrainingAttendance.vue           | 55 +-------------
 2 files changed, 77 insertions(+), 52 deletions(-)
 create mode 100644 src/components/Dialogs/AddStudentDialog.vue

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
new file mode 100644
index 0000000..9bcb880
--- /dev/null
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -0,0 +1,74 @@
+<template>
+  <BaseDialog :show-dialog="showDialog" dialog-title="Add Student">
+    <v-card-text :style="{ height: '400px' }">
+
+      <UserSelect
+        :force-error="false"
+        :force-error-message="
+          []
+        "
+        multiple
+        ref="userSelect"
+        class="course-select-dialog-user-select"
+      />
+      <label
+        v-if="true"
+        class="text-left w-100 d-block text--white available-seats-label"
+        style="
+          width: 81%;
+          display: block;
+          position: relative;
+          left: 33px;
+          bottom: 11px;
+        "
+        >You have
+        available seat reservations</label
+      >
+    </v-card-text>
+
+    <template v-slot:actions>
+      <v-btn
+        color=""
+        class="primary"
+        :disabled="state.isAddingBusy"
+        text
+        @click="cancel()"
+      >
+        Cancel
+      </v-btn>
+      <v-btn
+        color="primary"
+        class="limegreen"
+        :loading="state.isAddingBusy"
+        text
+      >
+        Add To Course
+      </v-btn>
+    </template>
+  </BaseDialog>
+</template>
+<script>
+import UserSelect from "@/components/UserSelect";
+import BaseDialog from "@/components/Dialogs/BaseDialog";
+
+export default {
+  name: "TrainingAttendance",
+  components: { UserSelect, BaseDialog },
+  props: {
+    showDialog: Boolean,
+  },
+  data() {
+    return {
+      state: {
+        isAddingBusy: false
+      }
+    }
+  },
+  async mounted() {
+
+  },
+};
+</script>
+<style>
+
+</style>
diff --git a/src/components/Training/TrainingAttendance.vue b/src/components/Training/TrainingAttendance.vue
index 734f613..e66c148 100644
--- a/src/components/Training/TrainingAttendance.vue
+++ b/src/components/Training/TrainingAttendance.vue
@@ -114,62 +114,14 @@
         </v-btn>
       </div>
     </div>
-    <v-dialog
-      v-model="state.isAddingStudent"
-      persistent
-      no-click-animation
-      max-width="500px"
-    >
-      <v-card>
-        <v-card-title>
-          <span class="headline"
-            >Add new student to {{ currentTrainingCourse.name }}</span
-          >
-        </v-card-title>
+    <AddStudentDialog :show-dialog="state.isAddingStudent" />
 
-        <v-card-text>
-          <UserSelect
-            ref="userSelect"
-            v-model="studentToAdd"
-            :rules="[
-              inputRules.required,
-              inputRules.duplicates(
-                studentToAdd,
-                currentTrainingCourse.students,
-                'User is already a student of this training'
-              ),
-            ]"
-            multiple
-            required
-          />
-        </v-card-text>
 
-        <v-card-actions>
-          <v-spacer></v-spacer>
-          <v-btn
-            color=""
-            :disabled="state.isAddingBusy"
-            text
-            @click="cancelAddStudent()"
-          >
-            Cancel
-          </v-btn>
-          <v-btn
-            :disabled="state.isAddingDuplicate || !studentToAdd"
-            color="primary"
-            :loading="state.isAddingBusy"
-            text
-            @click="addStudent()"
-          >
-            Add Student
-          </v-btn>
-        </v-card-actions>
-      </v-card>
-    </v-dialog>
   </div>
 </template>
 
 <script>
+import AddStudentDialog from "@/components/Dialogs/AddStudentDialog";
 import TrainingService from "@/api/services/training";
 import { debounceTimeout } from "@/config/config";
 import inputRules from "@/utils/inputRules";
@@ -182,13 +134,12 @@ import {
 } from "@/config/table-constants";
 import { createDefaultPaginationParams } from "@/utils/tableHelpers";
 import moment from "moment";
-import UserSelect from "@/components/UserSelect";
 import { minimumWait } from "@/utils/timing";
 
 const minCheckBoxWaitTime = 500; // ms
 export default {
   name: "TrainingAttendance",
-  components: { UserSelect },
+  components: { AddStudentDialog },
   props: {
     trainingCourse: {
       type: Object,
-- 
GitLab


From c9c48414d19ae6289986991195cc2368cb3a5bdc Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 10:43:44 -0600
Subject: [PATCH 02/63] Change dialog title

---
 src/components/Dialogs/AddStudentDialog.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 9bcb880..a9abb40 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -1,5 +1,5 @@
 <template>
-  <BaseDialog :show-dialog="showDialog" dialog-title="Add Student">
+  <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
     <v-card-text :style="{ height: '400px' }">
 
       <UserSelect
-- 
GitLab


From de201ac5644440b4705af6933f6b1f03731d7727 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 10:59:08 -0600
Subject: [PATCH 03/63] refactor

---
 src/components/Dialogs/AddStudentDialog.vue   |  4 +-
 src/components/Dialogs/CourseSelectDialog.vue | 44 +++----------
 .../DialogComponents/UserSelectForCourse.vue  | 62 +++++++++++++++++++
 3 files changed, 72 insertions(+), 38 deletions(-)
 create mode 100644 src/components/Dialogs/DialogComponents/UserSelectForCourse.vue

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index a9abb40..35cf0c9 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -1,6 +1,6 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
-    <v-card-text :style="{ height: '400px' }">
+    <v-card-text :style="{ height: '200px' }">
 
       <UserSelect
         :force-error="false"
@@ -52,7 +52,7 @@ import UserSelect from "@/components/UserSelect";
 import BaseDialog from "@/components/Dialogs/BaseDialog";
 
 export default {
-  name: "TrainingAttendance",
+  name: "AddStudentDialog",
   components: { UserSelect, BaseDialog },
   props: {
     showDialog: Boolean,
diff --git a/src/components/Dialogs/CourseSelectDialog.vue b/src/components/Dialogs/CourseSelectDialog.vue
index f7193e5..0e76054 100644
--- a/src/components/Dialogs/CourseSelectDialog.vue
+++ b/src/components/Dialogs/CourseSelectDialog.vue
@@ -10,40 +10,12 @@
         multiple
       />
 
-      <v-text-field
-        ref="startDate"
-        v-model="courseStartDate"
-        prepend-icon="mdi-calendar-today"
-        label="Start Date"
-        hint="* Required"
-        :persistent-hint="true"
-      >
-      </v-text-field>
-      <UserSelect
-        :force-error="areNoAvailableSeats"
-        :force-error-message="
-          areNoAvailableSeats ? ['No more available seats'] : []
-        "
-        multiple
-        ref="userSelect"
-        class="course-select-dialog-user-select"
-        v-on:input="updateCourseMembers"
-        v-bind:value="internalSelectedMembers"
-      />
-      <label
-        v-if="isCourseSelected && !areNoAvailableSeats"
-        class="text-left w-100 d-block text--white available-seats-label"
-        style="
-          width: 81%;
-          display: block;
-          position: relative;
-          left: 33px;
-          bottom: 11px;
-        "
-        >You have
-        {{ availableSeatsInCourse - internalSelectedMembers.length }}
-        available seat reservations</label
-      >
+      <UserSelectForCourse :are-no-available-seats="areNoAvailableSeats"
+        :available-seats-in-course="availableSeatsInCourse"
+        :course-start-date="courseStartDate"
+        :update-course-members="updateCourseMembers"
+        :is-course-selected="isCourseSelected"
+       />
     </v-card-text>
 
     <template v-slot:actions>
@@ -81,7 +53,7 @@ import CourseSelect from "@/components/CourseSelect";
 import { setInputElementFocus } from "@/utils/events";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
 import CourseService from "@/api/services/training";
-import UserSelect from "@/components/UserSelect";
+import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
 
 export default {
   name: "CourseSelectDialog",
@@ -97,8 +69,8 @@ export default {
   },
   components: {
     BaseDialog,
-    UserSelect,
     CourseSelect,
+    UserSelectForCourse,
   },
   data: () => ({
     isUserSelectError: false,
diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
new file mode 100644
index 0000000..4b71c05
--- /dev/null
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -0,0 +1,62 @@
+<template>
+  <div>
+  <v-text-field
+    ref="startDate"
+    v-model="courseStartDate"
+    prepend-icon="mdi-calendar-today"
+    label="Start Date"
+    hint="* Required"
+    :persistent-hint="true"
+  >
+  </v-text-field>
+  <UserSelect
+    :force-error="areNoAvailableSeats"
+    :force-error-message="
+      areNoAvailableSeats ? ['No more available seats'] : []
+    "
+    multiple
+    ref="userSelect"
+    class="course-select-dialog-user-select"
+    v-on:input="updateCourseMembers"
+    v-bind:value="internalSelectedMembers"
+  />
+  <label
+    v-if="isCourseSelected && !areNoAvailableSeats"
+    class="text-left w-100 d-block text--white available-seats-label"
+    style="
+      width: 81%;
+      display: block;
+      position: relative;
+      left: 33px;
+      bottom: 11px;
+    "
+    >You have
+    {{ availableSeatsInCourse - internalSelectedMembers.length }}
+    available seat reservations</label
+  >
+</div>
+</template>
+<script>
+import UserSelect from "@/components/UserSelect";
+
+export default {
+  name: "UserSelectForCourse",
+  components: { UserSelect },
+  props: {
+    areNoAvailableSeats: Boolean,
+    availableSeatsInCourse: Number,
+    courseStartDate: String,
+    updateCourseMembers: Function,
+    isCourseSelected: Boolean
+  },
+  data() {
+    return {
+      state: {
+        isAddingBusy: false
+      }
+    }
+  },
+};
+</script>
+<style>
+</style>
-- 
GitLab


From 8266388c04c4b6f279bfa625fca97a42ca857eee Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 11:32:41 -0600
Subject: [PATCH 04/63] Add Student Dialog looks good

---
 src/components/Dialogs/AddStudentDialog.vue   | 38 +++++----------
 .../DialogComponents/CourseNameDate.vue       | 43 +++++++++++++++++
 .../DialogComponents/UserSelectForCourse.vue  | 47 +++++++++++++------
 .../Dialogs/vue-component-boiler-plate.vue    | 22 +++++++++
 .../Training/TrainingAttendance.vue           |  7 +--
 5 files changed, 113 insertions(+), 44 deletions(-)
 create mode 100644 src/components/Dialogs/DialogComponents/CourseNameDate.vue
 create mode 100644 src/components/Dialogs/vue-component-boiler-plate.vue

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 35cf0c9..77b80a0 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -1,29 +1,9 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
-    <v-card-text :style="{ height: '200px' }">
-
-      <UserSelect
-        :force-error="false"
-        :force-error-message="
-          []
-        "
-        multiple
-        ref="userSelect"
-        class="course-select-dialog-user-select"
-      />
-      <label
-        v-if="true"
-        class="text-left w-100 d-block text--white available-seats-label"
-        style="
-          width: 81%;
-          display: block;
-          position: relative;
-          left: 33px;
-          bottom: 11px;
-        "
-        >You have
-        available seat reservations</label
-      >
+    <v-card-text :style="{ height: '300px', display: 'flex', 'flex-direction': 'column', 'justify-content':'flex-end', 'padding-bottom':'150px' }">
+      <CourseNameDate />
+      <div :style="{height:'10px'}"></div>
+      <UserSelectForCourse />
     </v-card-text>
 
     <template v-slot:actions>
@@ -48,15 +28,21 @@
   </BaseDialog>
 </template>
 <script>
-import UserSelect from "@/components/UserSelect";
 import BaseDialog from "@/components/Dialogs/BaseDialog";
+import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
+import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
 
 export default {
   name: "AddStudentDialog",
-  components: { UserSelect, BaseDialog },
+  components: { BaseDialog, UserSelectForCourse, CourseNameDate },
   props: {
     showDialog: Boolean,
   },
+  methods: {
+    cancel() {
+      this.$emit('close-dialog');
+    }
+  },
   data() {
     return {
       state: {
diff --git a/src/components/Dialogs/DialogComponents/CourseNameDate.vue b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
new file mode 100644
index 0000000..2d346f3
--- /dev/null
+++ b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
@@ -0,0 +1,43 @@
+<template>
+  <div class="course-name-date">
+    <span class="course-name">{{courseName}}</span><span class="course-date">{{courseDate}}</span>
+  </div>
+</template>
+<script>
+
+export default {
+  name: "CourseNameDate",
+  components: { },
+  props: {
+    courseName: {
+      type: String,
+      default: "No Course Name"
+    },
+    courseDate: {
+      type: String,
+      default: "No Course Date"
+    }
+  },
+  data() {
+    return {
+    }
+  },
+};
+</script>
+<style scoped>
+.course-name-date {
+  display: flex;
+  flex-direction: row;
+  justify-content: space-between;
+  padding: 0 3em;
+}
+
+.course-name {
+  font-weight: bold;
+  color: white;
+}
+
+.course-date {
+  color: white;
+}
+</style>
diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index 4b71c05..c2312a6 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -1,14 +1,6 @@
 <template>
   <div>
-  <v-text-field
-    ref="startDate"
-    v-model="courseStartDate"
-    prepend-icon="mdi-calendar-today"
-    label="Start Date"
-    hint="* Required"
-    :persistent-hint="true"
-  >
-  </v-text-field>
+
   <UserSelect
     :force-error="areNoAvailableSeats"
     :force-error-message="
@@ -21,7 +13,7 @@
     v-bind:value="internalSelectedMembers"
   />
   <label
-    v-if="isCourseSelected && !areNoAvailableSeats"
+    v-if="availableSeatsInCourse && isCourseSelected && !areNoAvailableSeats"
     class="text-left w-100 d-block text--white available-seats-label"
     style="
       width: 81%;
@@ -43,11 +35,26 @@ export default {
   name: "UserSelectForCourse",
   components: { UserSelect },
   props: {
-    areNoAvailableSeats: Boolean,
-    availableSeatsInCourse: Number,
-    courseStartDate: String,
-    updateCourseMembers: Function,
-    isCourseSelected: Boolean
+    areNoAvailableSeats: {
+      type: Boolean,
+      default: true
+    },
+    availableSeatsInCourse: {
+      type: Number,
+      default: 0
+    },
+    courseStartDate: {
+      type: String,
+      default: 'Select Course'
+    },
+    updateCourseMembers: {
+      type: Function,
+      default: () => {}
+    },
+    isCourseSelected: {
+      type: Boolean,
+      default: false
+    }
   },
   data() {
     return {
@@ -56,6 +63,16 @@ export default {
       }
     }
   },
+  created() {
+    console.log('test');
+    // console.log({
+    //   areNoAvailableSeats: this.areNoAvailableSeats,
+    //   availableSeatsInCourse: this.availableSeatsInCourse,
+    //   courseStartDate: this.courseStartDate,
+    //   updateCourseMembers: this.updateCourseMembers,
+    //   isCourseSelected: this.isCourseSelected
+    // });
+  },
 };
 </script>
 <style>
diff --git a/src/components/Dialogs/vue-component-boiler-plate.vue b/src/components/Dialogs/vue-component-boiler-plate.vue
new file mode 100644
index 0000000..22f812d
--- /dev/null
+++ b/src/components/Dialogs/vue-component-boiler-plate.vue
@@ -0,0 +1,22 @@
+<template>
+
+</template>
+<script>
+
+export default {
+  name: "CourseNameDate",
+  components: { },
+  props: {
+  },
+  data() {
+    return {
+    }
+  },
+  created() {
+
+  },
+};
+</script>
+<style>
+
+</style>
diff --git a/src/components/Training/TrainingAttendance.vue b/src/components/Training/TrainingAttendance.vue
index e66c148..7c2f662 100644
--- a/src/components/Training/TrainingAttendance.vue
+++ b/src/components/Training/TrainingAttendance.vue
@@ -114,9 +114,7 @@
         </v-btn>
       </div>
     </div>
-    <AddStudentDialog :show-dialog="state.isAddingStudent" />
-
-
+    <AddStudentDialog :show-dialog="state.isAddingStudent" v-on:close-dialog="closeAddStudentDialog" />
   </div>
 </template>
 
@@ -171,6 +169,9 @@ export default {
     await this.fetchData();
   },
   methods: {
+    closeAddStudentDialog() {
+      this.state.isAddingStudent = false;
+    },
     fetchDebounced() {
       // cancel pending call
       clearTimeout(this._timerId);
-- 
GitLab


From 482081e116192f09052a3128216f2f2e0a275345 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 11:36:55 -0600
Subject: [PATCH 05/63] Fix dialog

---
 src/components/Dialogs/CourseSelectDialog.vue | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/components/Dialogs/CourseSelectDialog.vue b/src/components/Dialogs/CourseSelectDialog.vue
index 0e76054..74cbc56 100644
--- a/src/components/Dialogs/CourseSelectDialog.vue
+++ b/src/components/Dialogs/CourseSelectDialog.vue
@@ -9,7 +9,15 @@
         "
         multiple
       />
-
+      <v-text-field
+        ref="startDate"
+        v-model="courseStartDate"
+        prepend-icon="mdi-calendar-today"
+        label="Start Date"
+        hint="* Required"
+        :persistent-hint="true"
+      ></v-text-field>
+      <div :style="{height:'40px'}"></div>
       <UserSelectForCourse :are-no-available-seats="areNoAvailableSeats"
         :available-seats-in-course="availableSeatsInCourse"
         :course-start-date="courseStartDate"
-- 
GitLab


From fbc1193cd67c5b763f2cc21ded97bf7d57a61ccb Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 11:42:05 -0600
Subject: [PATCH 06/63] default true

---
 src/components/Dialogs/DialogComponents/UserSelectForCourse.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index c2312a6..8fed2dd 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -37,7 +37,7 @@ export default {
   props: {
     areNoAvailableSeats: {
       type: Boolean,
-      default: true
+      default: false
     },
     availableSeatsInCourse: {
       type: Number,
-- 
GitLab


From 0fb6c44b001a58d3a0d6176ac885364429b2d027 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 12:01:09 -0600
Subject: [PATCH 07/63] course name date

---
 src/components/Dialogs/AddStudentDialog.vue   | 24 +++++++++++++++----
 .../DialogComponents/CourseNameDate.vue       |  2 ++
 2 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 77b80a0..324c22e 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -1,9 +1,15 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
     <v-card-text :style="{ height: '300px', display: 'flex', 'flex-direction': 'column', 'justify-content':'flex-end', 'padding-bottom':'150px' }">
-      <CourseNameDate />
+      <CourseNameDate
+          :course-name="course.name"
+          :course-date="course.startDate"
+       />
       <div :style="{height:'10px'}"></div>
-      <UserSelectForCourse />
+      <UserSelectForCourse :are-no-available-seats="areNoAvailableSeats"
+      :available-seats-in-course="availableSeatsInCourse"
+      :update-course-members="updateCourseMembers"
+      :is-course-selected="isCourseSelected"  />
     </v-card-text>
 
     <template v-slot:actions>
@@ -31,12 +37,14 @@
 import BaseDialog from "@/components/Dialogs/BaseDialog";
 import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
 import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
+import CourseService from "@/api/services/training";
 
 export default {
   name: "AddStudentDialog",
   components: { BaseDialog, UserSelectForCourse, CourseNameDate },
   props: {
     showDialog: Boolean,
+    courseId: Number
   },
   methods: {
     cancel() {
@@ -47,12 +55,18 @@ export default {
     return {
       state: {
         isAddingBusy: false
-      }
+      },
+      course: null
     }
   },
-  async mounted() {
-
+  computed: {
+    areNoAvailableSeats() {
+      return false;
+    },
   },
+  async created() {
+    this.course = await CourseService.getCourse(1);
+  }
 };
 </script>
 <style>
diff --git a/src/components/Dialogs/DialogComponents/CourseNameDate.vue b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
index 2d346f3..1c0f4d6 100644
--- a/src/components/Dialogs/DialogComponents/CourseNameDate.vue
+++ b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
@@ -33,11 +33,13 @@ export default {
 }
 
 .course-name {
+  font-size: 15px;
   font-weight: bold;
   color: white;
 }
 
 .course-date {
+  font-size: 15px;
   color: white;
 }
 </style>
-- 
GitLab


From 2a853aa2c175e4a8fbd9e134825836a5e3746865 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 12:09:52 -0600
Subject: [PATCH 08/63] missing props data

---
 src/components/Dialogs/AddStudentDialog.vue   | 25 ++++++++++++++++---
 .../DialogComponents/UserSelectForCourse.vue  |  3 ++-
 2 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 324c22e..655d02d 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -9,7 +9,7 @@
       <UserSelectForCourse :are-no-available-seats="areNoAvailableSeats"
       :available-seats-in-course="availableSeatsInCourse"
       :update-course-members="updateCourseMembers"
-      :is-course-selected="isCourseSelected"  />
+      :is-course-selected="true"  />
     </v-card-text>
 
     <template v-slot:actions>
@@ -47,6 +47,9 @@ export default {
     courseId: Number
   },
   methods: {
+    updateCourseMembers(members) {
+      this.courseMembers = members;
+    },
     cancel() {
       this.$emit('close-dialog');
     }
@@ -56,12 +59,28 @@ export default {
       state: {
         isAddingBusy: false
       },
-      course: null
+      course: {
+        name: '',
+        startDate: '',
+      },
+      courseMembers: [],
     }
   },
   computed: {
     areNoAvailableSeats() {
-      return false;
+      return this.availableSeatsInCourse <= 0;
+    },
+    availableSeatsInCourse() {
+      const course = this.course.capacity;
+      let seats = 0;
+      if (!course) {
+        seats = 0;
+      } else {
+        seats = course.capacity;
+      }
+      const addingUsersLength = this.courseMembers.length;
+      seats -= addingUsersLength;
+      return seats;
     },
   },
   async created() {
diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index 8fed2dd..ed23ecc 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -60,7 +60,8 @@ export default {
     return {
       state: {
         isAddingBusy: false
-      }
+      },
+      internalSelectedMembers: []
     }
   },
   created() {
-- 
GitLab


From ca80aeb63b1fbcd7acdc4f844701ee8b0364d551 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 12:13:51 -0600
Subject: [PATCH 09/63] label visible

---
 src/components/Dialogs/DialogComponents/UserSelectForCourse.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index ed23ecc..c61b5ec 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -13,7 +13,7 @@
     v-bind:value="internalSelectedMembers"
   />
   <label
-    v-if="availableSeatsInCourse && isCourseSelected && !areNoAvailableSeats"
+    v-if="isCourseSelected && !areNoAvailableSeats"
     class="text-left w-100 d-block text--white available-seats-label"
     style="
       width: 81%;
-- 
GitLab


From b29ac7adeff940bda320e95ca5d951fe193baa0b Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 13:43:02 -0600
Subject: [PATCH 10/63] Refactor

---
 src/components/Dialogs/AddStudentDialog.vue      | 16 ++++++++--------
 .../DialogComponents/UserSelectForCourse.vue     |  2 +-
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 655d02d..82c88fa 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -71,15 +71,15 @@ export default {
       return this.availableSeatsInCourse <= 0;
     },
     availableSeatsInCourse() {
-      const course = this.course.capacity;
-      let seats = 0;
-      if (!course) {
-        seats = 0;
-      } else {
-        seats = course.capacity;
+      let seats = this.course.capacity;
+      console.log('capacity is ', seats);
+      if (isNaN(seats)) {
+        console.info('seats is not a number');
+        seats = 10;
+      }
+      if (!isNaN(this.courseMembers.length)) {
+        seats -= this.courseMembers.length;
       }
-      const addingUsersLength = this.courseMembers.length;
-      seats -= addingUsersLength;
       return seats;
     },
   },
diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index c61b5ec..597691b 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -23,7 +23,7 @@
       bottom: 11px;
     "
     >You have
-    {{ availableSeatsInCourse - internalSelectedMembers.length }}
+    {{ availableSeatsInCourse }}
     available seat reservations</label
   >
 </div>
-- 
GitLab


From 2e350e4bce4bf68d302c318db72fbee0d80fc89d Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 13:43:50 -0600
Subject: [PATCH 11/63] remove console logs

---
 src/components/Dialogs/AddStudentDialog.vue | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 82c88fa..f01f640 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -72,9 +72,7 @@ export default {
     },
     availableSeatsInCourse() {
       let seats = this.course.capacity;
-      console.log('capacity is ', seats);
       if (isNaN(seats)) {
-        console.info('seats is not a number');
         seats = 10;
       }
       if (!isNaN(this.courseMembers.length)) {
-- 
GitLab


From 15e1c43ca66088aa75896c30bf3f947e5bc2a8f1 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 13:52:11 -0600
Subject: [PATCH 12/63] less than 0 error

---
 src/components/Dialogs/AddStudentDialog.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index f01f640..27102d8 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -68,7 +68,7 @@ export default {
   },
   computed: {
     areNoAvailableSeats() {
-      return this.availableSeatsInCourse <= 0;
+      return this.availableSeatsInCourse < 0;
     },
     availableSeatsInCourse() {
       let seats = this.course.capacity;
-- 
GitLab


From 984a146581f77c489284fb30b3c02e553fc6a2c1 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 14:00:37 -0600
Subject: [PATCH 13/63] not-operable styling

---
 src/components/Dialogs/AddStudentDialog.vue | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 27102d8..c73f474 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -23,6 +23,8 @@
         Cancel
       </v-btn>
       <v-btn
+        :disabled="areNoAvailableSeats"
+        :class="{ 'not-operable': areNoAvailableSeats }"
         color="primary"
         class="limegreen"
         :loading="state.isAddingBusy"
-- 
GitLab


From 27b5956d954a7a32ed2340898110f479944027b6 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 14:04:44 -0600
Subject: [PATCH 14/63] canSubmit

---
 src/components/Dialogs/AddStudentDialog.vue | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index c73f474..9643369 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -23,8 +23,8 @@
         Cancel
       </v-btn>
       <v-btn
-        :disabled="areNoAvailableSeats"
-        :class="{ 'not-operable': areNoAvailableSeats }"
+        :disabled="!canSubmit"
+        :class="{ 'not-operable': !canSubmit }"
         color="primary"
         class="limegreen"
         :loading="state.isAddingBusy"
@@ -82,6 +82,12 @@ export default {
       }
       return seats;
     },
+    canSubmit() {
+      console.log({ courseMembers: this.courseMembers })
+      const hasCourseMembers = this.courseMembers.length > 0;
+      console.log({hasCourseMembers});
+      return !this.areNoAvailableSeats && hasCourseMembers;
+    }
   },
   async created() {
     this.course = await CourseService.getCourse(1);
-- 
GitLab


From 6e9abb71a2f391e2bac2574612cd0283951f60ad Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 14:05:18 -0600
Subject: [PATCH 15/63] clean up

---
 src/components/Dialogs/AddStudentDialog.vue | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 9643369..9b12b27 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -83,10 +83,7 @@ export default {
       return seats;
     },
     canSubmit() {
-      console.log({ courseMembers: this.courseMembers })
-      const hasCourseMembers = this.courseMembers.length > 0;
-      console.log({hasCourseMembers});
-      return !this.areNoAvailableSeats && hasCourseMembers;
+      return !this.areNoAvailableSeats && this.courseMembers.length > 0;
     }
   },
   async created() {
-- 
GitLab


From dc5421c59a22a38b4c139caad8f9ce07830a2aa8 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 14:09:25 -0600
Subject: [PATCH 16/63] clean up

---
 .../Dialogs/DialogComponents/UserSelectForCourse.vue   | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index 597691b..272795b 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -64,16 +64,6 @@ export default {
       internalSelectedMembers: []
     }
   },
-  created() {
-    console.log('test');
-    // console.log({
-    //   areNoAvailableSeats: this.areNoAvailableSeats,
-    //   availableSeatsInCourse: this.availableSeatsInCourse,
-    //   courseStartDate: this.courseStartDate,
-    //   updateCourseMembers: this.updateCourseMembers,
-    //   isCourseSelected: this.isCourseSelected
-    // });
-  },
 };
 </script>
 <style>
-- 
GitLab


From 546996dddf9f1b10249774bee2b685a5ba0a7488 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 27 Apr 2021 14:31:23 -0600
Subject: [PATCH 17/63] currentCourseId

---
 src/components/Dialogs/AddStudentDialog.vue    | 2 +-
 src/components/Training/TrainingAttendance.vue | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentDialog.vue
index 9b12b27..97fa3be 100644
--- a/src/components/Dialogs/AddStudentDialog.vue
+++ b/src/components/Dialogs/AddStudentDialog.vue
@@ -87,7 +87,7 @@ export default {
     }
   },
   async created() {
-    this.course = await CourseService.getCourse(1);
+    this.course = await CourseService.getCourse(this.courseId);
   }
 };
 </script>
diff --git a/src/components/Training/TrainingAttendance.vue b/src/components/Training/TrainingAttendance.vue
index 7c2f662..25ed41c 100644
--- a/src/components/Training/TrainingAttendance.vue
+++ b/src/components/Training/TrainingAttendance.vue
@@ -114,7 +114,7 @@
         </v-btn>
       </div>
     </div>
-    <AddStudentDialog :show-dialog="state.isAddingStudent" v-on:close-dialog="closeAddStudentDialog" />
+    <AddStudentDialog :course-id="currentCourseId" :show-dialog="state.isAddingStudent" v-on:close-dialog="closeAddStudentDialog" />
   </div>
 </template>
 
-- 
GitLab


From ced71b7976579ca51c8204dac2dd5891035a2b96 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 11:26:03 -0600
Subject: [PATCH 18/63] Refactor

---
 ...alog.vue => AddStudentsToCourseDialog.vue} |   0
 .../DialogComponents/UserSelectForCourse.vue  |   7 +-
 ...ue => SelectAddStudentsToCourseDialog.vue} | 111 +++++++-----------
 .../Training/TrainingAttendance.vue           |   6 +-
 src/views/Team.vue                            |   6 +-
 5 files changed, 51 insertions(+), 79 deletions(-)
 rename src/components/Dialogs/{AddStudentDialog.vue => AddStudentsToCourseDialog.vue} (100%)
 rename src/components/Dialogs/{CourseSelectDialog.vue => SelectAddStudentsToCourseDialog.vue} (53%)

diff --git a/src/components/Dialogs/AddStudentDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
similarity index 100%
rename from src/components/Dialogs/AddStudentDialog.vue
rename to src/components/Dialogs/AddStudentsToCourseDialog.vue
diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index 272795b..ec8512b 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -10,7 +10,7 @@
     ref="userSelect"
     class="course-select-dialog-user-select"
     v-on:input="updateCourseMembers"
-    v-bind:value="internalSelectedMembers"
+    v-bind:value="selectedMembers"
   />
   <label
     v-if="isCourseSelected && !areNoAvailableSeats"
@@ -35,6 +35,10 @@ export default {
   name: "UserSelectForCourse",
   components: { UserSelect },
   props: {
+    selectedMembers: {
+      type: Array,
+      default: () => ([]),
+    },
     areNoAvailableSeats: {
       type: Boolean,
       default: false
@@ -61,7 +65,6 @@ export default {
       state: {
         isAddingBusy: false
       },
-      internalSelectedMembers: []
     }
   },
 };
diff --git a/src/components/Dialogs/CourseSelectDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
similarity index 53%
rename from src/components/Dialogs/CourseSelectDialog.vue
rename to src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index 74cbc56..d07adf3 100644
--- a/src/components/Dialogs/CourseSelectDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -4,14 +4,11 @@
       <CourseSelect
         v-on:input="updateCourseSelected"
         ref="courseSelect"
-        v-model="
-          state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo
-        "
         multiple
       />
       <v-text-field
         ref="startDate"
-        v-model="courseStartDate"
+        v-model="course.startDate"
         prepend-icon="mdi-calendar-today"
         label="Start Date"
         hint="* Required"
@@ -20,9 +17,10 @@
       <div :style="{height:'40px'}"></div>
       <UserSelectForCourse :are-no-available-seats="areNoAvailableSeats"
         :available-seats-in-course="availableSeatsInCourse"
-        :course-start-date="courseStartDate"
+        :course-start-date="course.startDate"
         :update-course-members="updateCourseMembers"
         :is-course-selected="isCourseSelected"
+        :selected-members="courseMembers"
        />
     </v-card-text>
 
@@ -30,23 +28,20 @@
       <v-btn
         color=""
         class="primary"
-        :disabled="state.isAddingBusy"
+        :disabled="isAddingBusy"
         text
         @click="cancel()"
       >
         Cancel
       </v-btn>
       <v-btn
-        :disabled="!isCourseSelected"
+        :disabled="!canSubmit"
         color="primary"
         class="limegreen"
         :class="{
-          'not-operable':
-            !isCourseSelected ||
-            !internalSelectedMembers.length ||
-            areNoAvailableSeats,
+          'not-operable': !canSubmit,
         }"
-        :loading="state.isAddingBusy"
+        :loading="isAddingBusy"
         text
         @click="addSelectedUsersToSelectedCourses()"
       >
@@ -64,7 +59,7 @@ import CourseService from "@/api/services/training";
 import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
 
 export default {
-  name: "CourseSelectDialog",
+  name: "SelectAddStudentsToCourseDialog",
   props: {
     addBusy: Boolean,
     showDialog: Boolean,
@@ -81,54 +76,31 @@ export default {
     UserSelectForCourse,
   },
   data: () => ({
+    course: { name: "", startDate: "", },
+    courseMembers: [],
     isUserSelectError: false,
     userSelectErrorMessage: [],
-    availableSeatsInCourse: 0,
-    internalSelectedMembers: [],
-    courseStartDate: "",
-    state: {
-      /* Feature: Adding User To Course
-      controls the state of the adding user to selected course dialog */
-      addToSelectedCourseFeature: {
-        //for controlling dialog
-        ui: {
-          //dialog open/close
-          isAdding: false,
-          //allows the user to click the button to submit
-          isAddingDuplicate: false,
-          //loading
-          isAddingBusy: false,
-        },
-        //data for CourseSelect component
-        data: {
-          coursesToAddSelectedUsersTo: null,
-        },
-      },
-    },
+    isAdding: false,
+    isAddingDuplicate: false,
+    isAddingBusy: false,
   }),
   methods: {
     updateCourseMembers(members) {
-      this.internalSelectedMembers = members;
+      this.courseMembers = members;
+      this.$emit('update-selected-members', members);
     },
-    async updateCourseSelected(model) {
-      const course = model[0];
-      if (course) {
-        this.courseStartDate = course.startDate;
-        const courseRegistrations = await CourseService.getCourseRegistrationsById(
-          course.id
-        );
-        const capacity = course.capacity,
-          registrations = courseRegistrations.registrations;
-        const seatsLeft = capacity - registrations.length;
-        this.availableSeatsInCourse = seatsLeft;
+    async updateCourseSelected(courses) {
+      if (Array.isArray(courses) && courses.length > 0) {
+        this.course = courses[0];
+      } else {
+        this.course = { name: "", startDate: "", };
       }
     },
     async addSelectedUsersToSelectedCourses() {
-      this.state.addToSelectedCourseFeature.ui.isAddingBusy = true;
+      this.isAddingBusy = true;
       try {
         await CourseService.addStudentsToCourses(
-          this.state.addToSelectedCourseFeature.data
-            .coursesToAddSelectedUsersTo,
+          this.course,
           this.selectedMembers
         );
       } catch (error) {
@@ -137,9 +109,9 @@ export default {
         this.$store.commit(SET_ERROR_DIALOG, true);
       } finally {
         this.$refs.courseSelect && this.$refs.courseSelect.clear();
-        this.state.addToSelectedCourseFeature.ui.isAddingBusy = false;
-        this.state.addToSelectedCourseFeature.ui.isAdding = false;
-        this.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo = null;
+        this.isAddingBusy = false;
+        this.isAdding = false;
+        this.course = null;
         this.selectedMembers = [];
         this.$emit("close-dialog");
       }
@@ -157,26 +129,23 @@ export default {
   },
   computed: {
     areNoAvailableSeats() {
-      return (
-        this.isCourseSelected &&
-        this.availableSeatsInCourse - this.internalSelectedMembers.length < 0
-      );
+      return this.availableSeatsInCourse < 0;
     },
-    isCourseSelected() {
-      return (
-        Array.isArray(
-          this.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo
-        ) &&
-        this.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo
-          .length > 0
-      );
+    availableSeatsInCourse() {
+      let seats = this.course.capacity;
+      if (isNaN(seats)) {
+        seats = 10;
+      }
+      if (!isNaN(this.courseMembers.length)) {
+        seats -= this.courseMembers.length;
+      }
+      return seats;
+    },
+    canSubmit() {
+      return this.isCourseSelected && !this.areNoAvailableSeats && this.courseMembers.length > 0;
     },
-    areMultipleCoursesSelected() {
-      return (
-        this.isCourseSelected &&
-        this.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo
-          .length > 1
-      );
+    isCourseSelected() {
+      return this.course.name !== "";
     },
   },
 };
diff --git a/src/components/Training/TrainingAttendance.vue b/src/components/Training/TrainingAttendance.vue
index 25ed41c..cd8e3ac 100644
--- a/src/components/Training/TrainingAttendance.vue
+++ b/src/components/Training/TrainingAttendance.vue
@@ -114,12 +114,12 @@
         </v-btn>
       </div>
     </div>
-    <AddStudentDialog :course-id="currentCourseId" :show-dialog="state.isAddingStudent" v-on:close-dialog="closeAddStudentDialog" />
+    <AddStudentsToCourseDialog :course-id="currentCourseId" :show-dialog="state.isAddingStudent" v-on:close-dialog="closeAddStudentDialog" />
   </div>
 </template>
 
 <script>
-import AddStudentDialog from "@/components/Dialogs/AddStudentDialog";
+import AddStudentsToCourseDialog from "@/components/Dialogs/AddStudentsToCourseDialog";
 import TrainingService from "@/api/services/training";
 import { debounceTimeout } from "@/config/config";
 import inputRules from "@/utils/inputRules";
@@ -137,7 +137,7 @@ import { minimumWait } from "@/utils/timing";
 const minCheckBoxWaitTime = 500; // ms
 export default {
   name: "TrainingAttendance",
-  components: { AddStudentDialog },
+  components: { AddStudentsToCourseDialog },
   props: {
     trainingCourse: {
       type: Object,
diff --git a/src/views/Team.vue b/src/views/Team.vue
index cda3b92..723d762 100644
--- a/src/views/Team.vue
+++ b/src/views/Team.vue
@@ -125,7 +125,7 @@
             <!--
               COURSE SELECT DIALOG
             -->
-            <CourseSelectDialog
+            <SelectAddStudentsToCourseDialog
               :style="{ height: '600px' }"
               ref="courseSelectDialog"
               v-bind:selected-members="selectedMembers"
@@ -299,7 +299,7 @@
 </template>
 <script>
 import Vue from "vue";
-import CourseSelectDialog from "@/components/Dialogs/CourseSelectDialog";
+import SelectAddStudentsToCourseDialog from "@/components/Dialogs/SelectAddStudentsToCourseDialog";
 import TeamService from "@/api/services/team";
 import UserSelect from "@/components/UserSelect";
 import AddTeam from "@/components/AddTeam";
@@ -313,7 +313,7 @@ import BaseDialog from "@/components/Dialogs/BaseDialog";
 export default {
   components: {
     UserSelect,
-    CourseSelectDialog,
+    SelectAddStudentsToCourseDialog,
     AddTeam,
     ErrorMessage,
     BaseDialog,
-- 
GitLab


From bef24159cebd9fef9a47dafd75faad6b3e69adba Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 12:15:56 -0600
Subject: [PATCH 19/63] subtract registrants

---
 .../Dialogs/SelectAddStudentsToCourseDialog.vue      | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index d07adf3..79f0375 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -77,6 +77,7 @@ export default {
   },
   data: () => ({
     course: { name: "", startDate: "", },
+    courseRegistrations: [],
     courseMembers: [],
     isUserSelectError: false,
     userSelectErrorMessage: [],
@@ -92,6 +93,10 @@ export default {
     async updateCourseSelected(courses) {
       if (Array.isArray(courses) && courses.length > 0) {
         this.course = courses[0];
+        if (!isNaN(this.course.id)) {
+          const result = await CourseService.getCourseRegistrationsById(this.course.id);
+          this.courseRegistrations = result.registrations;
+        }
       } else {
         this.course = { name: "", startDate: "", };
       }
@@ -100,7 +105,7 @@ export default {
       this.isAddingBusy = true;
       try {
         await CourseService.addStudentsToCourses(
-          this.course,
+          [this.course],
           this.selectedMembers
         );
       } catch (error) {
@@ -111,8 +116,8 @@ export default {
         this.$refs.courseSelect && this.$refs.courseSelect.clear();
         this.isAddingBusy = false;
         this.isAdding = false;
-        this.course = null;
-        this.selectedMembers = [];
+        this.course = { name: "", startDate: "", };
+        this.updateCourseMembers([]);
         this.$emit("close-dialog");
       }
     },
@@ -139,6 +144,7 @@ export default {
       if (!isNaN(this.courseMembers.length)) {
         seats -= this.courseMembers.length;
       }
+      seats -= this.courseRegistrations.length;
       return seats;
     },
     canSubmit() {
-- 
GitLab


From 6c684f5761457b2723ffd589eb1c076005f73c71 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 12:24:53 -0600
Subject: [PATCH 20/63] Refactor

---
 .../Dialogs/AddStudentsToCourseDialog.vue     | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index 97fa3be..bdf5b3f 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -29,6 +29,7 @@
         class="limegreen"
         :loading="state.isAddingBusy"
         text
+        @click="addSelectedUsersToSelectedCourses"
       >
         Add To Course
       </v-btn>
@@ -40,6 +41,7 @@ import BaseDialog from "@/components/Dialogs/BaseDialog";
 import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
 import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
 import CourseService from "@/api/services/training";
+import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
 
 export default {
   name: "AddStudentDialog",
@@ -49,6 +51,26 @@ export default {
     courseId: Number
   },
   methods: {
+    async addSelectedUsersToSelectedCourses() {
+      this.isAddingBusy = true;
+      try {
+        await CourseService.addStudents(
+          this.courseId,
+          this.courseMembers
+        );
+      } catch (error) {
+        console.error("!add courses error!", error);
+        this.$store.commit(SET_ERROR_MESSAGE, error);
+        this.$store.commit(SET_ERROR_DIALOG, true);
+      } finally {
+        this.$refs.courseSelect && this.$refs.courseSelect.clear();
+        this.isAddingBusy = false;
+        this.isAdding = false;
+        this.course = { name: "", startDate: "", };
+        this.updateCourseMembers([]);
+        this.$emit("close-dialog");
+      }
+    },
     updateCourseMembers(members) {
       this.courseMembers = members;
     },
-- 
GitLab


From 12990096ea25caa7b1d556c3ed05152e627a1a7d Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 12:29:17 -0600
Subject: [PATCH 21/63] Refactor

---
 src/components/Dialogs/AddStudentsToCourseDialog.vue | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index bdf5b3f..0297a08 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -87,6 +87,7 @@ export default {
         name: '',
         startDate: '',
       },
+      courseRegistrations: [],
       courseMembers: [],
     }
   },
@@ -102,6 +103,7 @@ export default {
       if (!isNaN(this.courseMembers.length)) {
         seats -= this.courseMembers.length;
       }
+      seats -= this.courseRegistrations.length;
       return seats;
     },
     canSubmit() {
@@ -110,6 +112,8 @@ export default {
   },
   async created() {
     this.course = await CourseService.getCourse(this.courseId);
+    const result = await CourseService.getCourseRegistrationsById(this.course.id);
+    this.courseRegistrations = result.registrations;
   }
 };
 </script>
-- 
GitLab


From 3472a6681b75d667c3dd188eaba4a4ee29760a0c Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 13:30:51 -0600
Subject: [PATCH 22/63] clean up

---
 ...=> SelectAddStudentToCourseDialog.spec.js} |  4 +-
 .../Training/TrainingAttendance.spec.js       | 92 -------------------
 2 files changed, 2 insertions(+), 94 deletions(-)
 rename tests/unit/components/Dialogs/{CourseSelectDialog.spec.js => SelectAddStudentToCourseDialog.spec.js} (96%)

diff --git a/tests/unit/components/Dialogs/CourseSelectDialog.spec.js b/tests/unit/components/Dialogs/SelectAddStudentToCourseDialog.spec.js
similarity index 96%
rename from tests/unit/components/Dialogs/CourseSelectDialog.spec.js
rename to tests/unit/components/Dialogs/SelectAddStudentToCourseDialog.spec.js
index eb9107f..28ef885 100644
--- a/tests/unit/components/Dialogs/CourseSelectDialog.spec.js
+++ b/tests/unit/components/Dialogs/SelectAddStudentToCourseDialog.spec.js
@@ -1,7 +1,7 @@
 import { createLocalVue, shallowMount } from "@vue/test-utils";
 import Vuetify from "vuetify";
 import Vuex from "vuex";
-import CourseSelectDialog from "@/components/Dialogs/CourseSelectDialog";
+import SelectAddStudentToCourseDialog from "@/components/Dialogs/SelectAddStudentToCourseDialog";
 
 const localVue = createLocalVue();
 localVue.use(Vuex);
@@ -9,7 +9,7 @@ localVue.use(Vuex);
 const testUser = { name: "test" };
 const testCourse = { name: "test" };
 
-describe("CourseSelectDialog", () => {
+describe("SelectAddStudentToCourseDialog", () => {
   // vuetify has to be mocked for the v-autocomplete to not crash
   let vuetify;
   beforeEach(() => {
diff --git a/tests/unit/components/Training/TrainingAttendance.spec.js b/tests/unit/components/Training/TrainingAttendance.spec.js
index bf79070..cf23a76 100644
--- a/tests/unit/components/Training/TrainingAttendance.spec.js
+++ b/tests/unit/components/Training/TrainingAttendance.spec.js
@@ -456,98 +456,6 @@ describe("TrainingAttendance", () => {
     expect(wrapper.vm.$store.commit).toHaveBeenCalledTimes(1);
   });
 
-  it("should add student", async () => {
-    TrainingService.getCourseRegistrationsById = jest
-      .fn()
-      .mockResolvedValue({ registrations: [] });
-    TrainingService.addStudents = jest.fn().mockResolvedValue([]);
-    const wrapper = shallowMount(TrainingAttendance, {
-      mocks: {
-        $store: {
-          state: {
-            error: {},
-          },
-          commit: jest.fn(),
-        },
-      },
-      data() {
-        return {
-          selectedUsers: [{ userId: 1 }, { userId: 2 }],
-          listItems: [{ userId: 1 }, { userId: 2 }, { userId: 3 }],
-        };
-      },
-      propsData,
-      localVue,
-      vuetify,
-    });
-    await flushPromises();
-
-    wrapper.vm.$refs.userSelect.clear = jest.fn();
-    await wrapper.vm.addStudent([{ id: 1 }]);
-    expect(TrainingService.addStudents).toHaveBeenCalledTimes(1);
-  });
-
-  it("addStudent should catch error", async () => {
-    TrainingService.getCourseRegistrationsById = jest
-      .fn()
-      .mockResolvedValue({ registrations: [] });
-    TrainingService.addStudents = new Error("error");
-    const wrapper = shallowMount(TrainingAttendance, {
-      mocks: {
-        $store: {
-          state: {
-            error: {},
-          },
-          commit: jest.fn(),
-        },
-      },
-      data() {
-        return {
-          selectedUsers: [{ userId: 1 }, { userId: 2 }],
-          listItems: [{ userId: 1 }, { userId: 2 }, { userId: 3 }],
-          studentToAdd: [{ id: 1 }],
-        };
-      },
-      propsData,
-      localVue,
-      vuetify,
-    });
-    await flushPromises();
-
-    wrapper.vm.$refs.userSelect.clear = jest.fn();
-    await wrapper.vm.addStudent();
-    expect(wrapper.vm.$store.commit).toHaveBeenCalledTimes(1);
-  });
-
-  it("should cancel add student", async () => {
-    TrainingService.getCourseRegistrationsById = jest
-      .fn()
-      .mockResolvedValue({ registrations: [] });
-    const wrapper = shallowMount(TrainingAttendance, {
-      mocks: {
-        $store: {
-          state: {
-            error: {},
-          },
-          commit: jest.fn(),
-        },
-      },
-      data() {
-        return {
-          selectedUsers: [{ userId: 1 }, { userId: 2 }],
-          listItems: [{ userId: 1 }, { userId: 2 }, { userId: 3 }],
-        };
-      },
-      propsData,
-      localVue,
-      vuetify,
-    });
-    await flushPromises();
-
-    wrapper.vm.$refs.userSelect.clear = jest.fn();
-    wrapper.vm.cancelAddStudent();
-    expect(wrapper.vm.$refs.userSelect.clear).toHaveBeenCalledTimes(1);
-  });
 
   it("should call fetchDebounced when options change", async () => {
     TrainingService.getCourseRegistrationsById = jest
-- 
GitLab


From 8f40c313ebbd7f4c4d5b1bfa38f55b513e91730d Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 13:58:41 -0600
Subject: [PATCH 23/63] rename files

---
 ...seDialog.spec.js => SelectAddStudentsToCourseDialog.spec.js} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename tests/unit/components/Dialogs/{SelectAddStudentToCourseDialog.spec.js => SelectAddStudentsToCourseDialog.spec.js} (99%)

diff --git a/tests/unit/components/Dialogs/SelectAddStudentToCourseDialog.spec.js b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
similarity index 99%
rename from tests/unit/components/Dialogs/SelectAddStudentToCourseDialog.spec.js
rename to tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
index 28ef885..ea68264 100644
--- a/tests/unit/components/Dialogs/SelectAddStudentToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
@@ -1,7 +1,7 @@
 import { createLocalVue, shallowMount } from "@vue/test-utils";
 import Vuetify from "vuetify";
 import Vuex from "vuex";
-import SelectAddStudentToCourseDialog from "@/components/Dialogs/SelectAddStudentToCourseDialog";
+import SelectAddStudentToCourseDialog from "@/components/Dialogs/SelectAddStudentsToCourseDialog";
 
 const localVue = createLocalVue();
 localVue.use(Vuex);
-- 
GitLab


From a51f1cd6c9d032fd254957e68eb2c6af266bd0dd Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 14:20:46 -0600
Subject: [PATCH 24/63] unit tests

---
 .../SelectAddStudentsToCourseDialog.spec.js   | 61 ++++---------------
 1 file changed, 13 insertions(+), 48 deletions(-)

diff --git a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
index ea68264..fdedd64 100644
--- a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
@@ -1,15 +1,15 @@
 import { createLocalVue, shallowMount } from "@vue/test-utils";
 import Vuetify from "vuetify";
 import Vuex from "vuex";
-import SelectAddStudentToCourseDialog from "@/components/Dialogs/SelectAddStudentsToCourseDialog";
+import SelectAddStudentsToCourseDialog from "@/components/Dialogs/SelectAddStudentsToCourseDialog";
 
 const localVue = createLocalVue();
 localVue.use(Vuex);
 
 const testUser = { name: "test" };
-const testCourse = { name: "test" };
+const testCourse = { name: "test", capacity: 100 };
 
-describe("SelectAddStudentToCourseDialog", () => {
+describe("SelectAddStudentsToCourseDialog", () => {
   // vuetify has to be mocked for the v-autocomplete to not crash
   let vuetify;
   beforeEach(() => {
@@ -20,7 +20,7 @@ describe("SelectAddStudentToCourseDialog", () => {
   });
 
   it("should update internal course members when selected members changes", async () => {
-    const wrapper = shallowMount(CourseSelectDialog, {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
         value: [],
         selectedMembers: [testUser, testUser, testUser, testUser],
@@ -34,7 +34,7 @@ describe("SelectAddStudentToCourseDialog", () => {
   });
 
   it("if areNoAvailableSeats is true then should show error on UserSelect", async () => {
-    const wrapper = shallowMount(CourseSelectDialog, {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
         value: [],
         selectedMembers: [testUser, testUser, testUser, testUser],
@@ -42,40 +42,15 @@ describe("SelectAddStudentToCourseDialog", () => {
       },
       vuetify,
     });
-    wrapper.vm.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo = [
-      testCourse,
-      testCourse,
-      testCourse,
-    ];
-    wrapper.vm.availableSeatsInCourse = 0;
+    wrapper.vm.course = testCourse;
     await wrapper.vm.$nextTick();
     const userSelectWrapper = wrapper.find(".user-select-component");
     expect(userSelectWrapper.exists()).toBe(false);
   });
 
-  it("availableSeatsInCourse should appear in label under UserSelect if course is selected", async () => {
-    const wrapper = shallowMount(CourseSelectDialog, {
-      propsData: {
-        value: [],
-        selectedMembers: [testUser, testUser, testUser, testUser],
-        showDialog: true,
-      },
-      vuetify,
-    });
-    wrapper.vm.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo = [
-      testCourse,
-      testCourse,
-      testCourse,
-    ];
-    wrapper.vm.availableSeatsInCourse = 10;
-    await wrapper.vm.$nextTick();
-    const userSelectWrapper = wrapper.find(".available-seats-label");
-    const text = userSelectWrapper.html();
-    expect(text.indexOf("available seat reservations") !== -1).toBe(true);
-  });
 
-  it("internalSelectedMembers should start as empty array", async () => {
-    const wrapper = shallowMount(CourseSelectDialog, {
+  it("courseMembers should start as empty array", async () => {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
         value: [],
         selectedMembers: [testUser, testUser, testUser, testUser],
@@ -83,21 +58,16 @@ describe("SelectAddStudentToCourseDialog", () => {
       },
       vuetify,
     });
-    wrapper.vm.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo = [
-      testCourse,
-      testCourse,
-      testCourse,
-    ];
-    wrapper.vm.availableSeatsInCourse = 10;
+    wrapper.vm.course = testCourse;
     await wrapper.vm.$nextTick();
     expect(
-      Array.isArray(wrapper.vm.internalSelectedMembers) &&
-        wrapper.vm.internalSelectedMembers.length === 0
+      Array.isArray(wrapper.vm.courseMembers) &&
+        wrapper.vm.courseMembers.length === 0
     ).toBe(true);
   });
 
   it("internalSelectedMembers should be equal to selectedMembers", async () => {
-    const wrapper = shallowMount(CourseSelectDialog, {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
         value: [],
         selectedMembers: [testUser, testUser, testUser, testUser],
@@ -105,12 +75,7 @@ describe("SelectAddStudentToCourseDialog", () => {
       },
       vuetify,
     });
-    wrapper.vm.state.addToSelectedCourseFeature.data.coursesToAddSelectedUsersTo = [
-      testCourse,
-      testCourse,
-      testCourse,
-    ];
-    wrapper.vm.availableSeatsInCourse = 10;
+    wrapper.vm.course = testCourse;
     await wrapper.vm.$nextTick();
     expect(
       wrapper.vm.selectedMembers == wrapper.vm.internalSelectedMembers
-- 
GitLab


From 160cee6a253e0d54730b5fcdf985a7921b2199ad Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 14:34:25 -0600
Subject: [PATCH 25/63] add unit tests

---
 .../Dialogs/DialogComponents/CourseNameDate.vue   | 15 +++++++--------
 .../SelectAddStudentsToCourseDialog.spec.js       |  1 -
 2 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/src/components/Dialogs/DialogComponents/CourseNameDate.vue b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
index 1c0f4d6..7262189 100644
--- a/src/components/Dialogs/DialogComponents/CourseNameDate.vue
+++ b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
@@ -1,26 +1,25 @@
 <template>
   <div class="course-name-date">
-    <span class="course-name">{{courseName}}</span><span class="course-date">{{courseDate}}</span>
+    <span class="course-name">{{ courseName }}</span
+    ><span class="course-date">{{ courseDate }}</span>
   </div>
 </template>
 <script>
-
 export default {
   name: "CourseNameDate",
-  components: { },
+  components: {},
   props: {
     courseName: {
       type: String,
-      default: "No Course Name"
+      default: "No Course Name",
     },
     courseDate: {
       type: String,
-      default: "No Course Date"
-    }
+      default: "No Course Date",
+    },
   },
   data() {
-    return {
-    }
+    return {};
   },
 };
 </script>
diff --git a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
index fdedd64..cdc32f1 100644
--- a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
@@ -48,7 +48,6 @@ describe("SelectAddStudentsToCourseDialog", () => {
     expect(userSelectWrapper.exists()).toBe(false);
   });
 
-
   it("courseMembers should start as empty array", async () => {
     const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
-- 
GitLab


From f15a9abbf9406ec2d447bf0c55dbc3a28819452c Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 14:36:50 -0600
Subject: [PATCH 26/63] prettier

---
 .../DialogComponents/UserSelectForCourse.vue  | 74 +++++++++----------
 1 file changed, 36 insertions(+), 38 deletions(-)

diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index ec8512b..509c5d4 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -1,32 +1,31 @@
 <template>
   <div>
-
-  <UserSelect
-    :force-error="areNoAvailableSeats"
-    :force-error-message="
-      areNoAvailableSeats ? ['No more available seats'] : []
-    "
-    multiple
-    ref="userSelect"
-    class="course-select-dialog-user-select"
-    v-on:input="updateCourseMembers"
-    v-bind:value="selectedMembers"
-  />
-  <label
-    v-if="isCourseSelected && !areNoAvailableSeats"
-    class="text-left w-100 d-block text--white available-seats-label"
-    style="
-      width: 81%;
-      display: block;
-      position: relative;
-      left: 33px;
-      bottom: 11px;
-    "
-    >You have
-    {{ availableSeatsInCourse }}
-    available seat reservations</label
-  >
-</div>
+    <UserSelect
+      :force-error="areNoAvailableSeats"
+      :force-error-message="
+        areNoAvailableSeats ? ['No more available seats'] : []
+      "
+      multiple
+      ref="userSelect"
+      class="course-select-dialog-user-select"
+      v-on:input="updateCourseMembers"
+      v-bind:value="selectedMembers"
+    />
+    <label
+      v-if="isCourseSelected && !areNoAvailableSeats"
+      class="text-left w-100 d-block text--white available-seats-label"
+      style="
+        width: 81%;
+        display: block;
+        position: relative;
+        left: 33px;
+        bottom: 11px;
+      "
+      >You have
+      {{ availableSeatsInCourse }}
+      available seat reservations</label
+    >
+  </div>
 </template>
 <script>
 import UserSelect from "@/components/UserSelect";
@@ -37,37 +36,36 @@ export default {
   props: {
     selectedMembers: {
       type: Array,
-      default: () => ([]),
+      default: () => [],
     },
     areNoAvailableSeats: {
       type: Boolean,
-      default: false
+      default: false,
     },
     availableSeatsInCourse: {
       type: Number,
-      default: 0
+      default: 0,
     },
     courseStartDate: {
       type: String,
-      default: 'Select Course'
+      default: "Select Course",
     },
     updateCourseMembers: {
       type: Function,
-      default: () => {}
+      default: () => {},
     },
     isCourseSelected: {
       type: Boolean,
-      default: false
-    }
+      default: false,
+    },
   },
   data() {
     return {
       state: {
-        isAddingBusy: false
+        isAddingBusy: false,
       },
-    }
+    };
   },
 };
 </script>
-<style>
-</style>
+<style></style>
-- 
GitLab


From 8c3db5a9e9cc4136d3a2f4fc7f276eae93b32718 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 14:38:57 -0600
Subject: [PATCH 27/63] prettier

---
 .../SelectAddStudentsToCourseDialog.vue       | 25 ++++++++++++-------
 .../Dialogs/vue-component-boiler-plate.vue    | 21 +++++-----------
 2 files changed, 22 insertions(+), 24 deletions(-)

diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index 79f0375..6b2e138 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -14,14 +14,15 @@
         hint="* Required"
         :persistent-hint="true"
       ></v-text-field>
-      <div :style="{height:'40px'}"></div>
-      <UserSelectForCourse :are-no-available-seats="areNoAvailableSeats"
+      <div :style="{ height: '40px' }"></div>
+      <UserSelectForCourse
+        :are-no-available-seats="areNoAvailableSeats"
         :available-seats-in-course="availableSeatsInCourse"
         :course-start-date="course.startDate"
         :update-course-members="updateCourseMembers"
         :is-course-selected="isCourseSelected"
         :selected-members="courseMembers"
-       />
+      />
     </v-card-text>
 
     <template v-slot:actions>
@@ -76,7 +77,7 @@ export default {
     UserSelectForCourse,
   },
   data: () => ({
-    course: { name: "", startDate: "", },
+    course: { name: "", startDate: "" },
     courseRegistrations: [],
     courseMembers: [],
     isUserSelectError: false,
@@ -88,17 +89,19 @@ export default {
   methods: {
     updateCourseMembers(members) {
       this.courseMembers = members;
-      this.$emit('update-selected-members', members);
+      this.$emit("update-selected-members", members);
     },
     async updateCourseSelected(courses) {
       if (Array.isArray(courses) && courses.length > 0) {
         this.course = courses[0];
         if (!isNaN(this.course.id)) {
-          const result = await CourseService.getCourseRegistrationsById(this.course.id);
+          const result = await CourseService.getCourseRegistrationsById(
+            this.course.id
+          );
           this.courseRegistrations = result.registrations;
         }
       } else {
-        this.course = { name: "", startDate: "", };
+        this.course = { name: "", startDate: "" };
       }
     },
     async addSelectedUsersToSelectedCourses() {
@@ -116,7 +119,7 @@ export default {
         this.$refs.courseSelect && this.$refs.courseSelect.clear();
         this.isAddingBusy = false;
         this.isAdding = false;
-        this.course = { name: "", startDate: "", };
+        this.course = { name: "", startDate: "" };
         this.updateCourseMembers([]);
         this.$emit("close-dialog");
       }
@@ -148,7 +151,11 @@ export default {
       return seats;
     },
     canSubmit() {
-      return this.isCourseSelected && !this.areNoAvailableSeats && this.courseMembers.length > 0;
+      return (
+        this.isCourseSelected &&
+        !this.areNoAvailableSeats &&
+        this.courseMembers.length > 0
+      );
     },
     isCourseSelected() {
       return this.course.name !== "";
diff --git a/src/components/Dialogs/vue-component-boiler-plate.vue b/src/components/Dialogs/vue-component-boiler-plate.vue
index 22f812d..d127bf8 100644
--- a/src/components/Dialogs/vue-component-boiler-plate.vue
+++ b/src/components/Dialogs/vue-component-boiler-plate.vue
@@ -1,22 +1,13 @@
-<template>
-
-</template>
+<template></template>
 <script>
-
 export default {
   name: "CourseNameDate",
-  components: { },
-  props: {
-  },
+  components: {},
+  props: {},
   data() {
-    return {
-    }
-  },
-  created() {
-
+    return {};
   },
+  created() {},
 };
 </script>
-<style>
-
-</style>
+<style></style>
-- 
GitLab


From bbda4c3c514bd699cbc455b4c21446c20ade2350 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 14:49:10 -0600
Subject: [PATCH 28/63] unit tests

---
 .../Dialogs/vue-component-boiler-plate.vue    | 13 -----
 .../Dialogs/AddStudentsToCourseDialog.spec.js | 21 +++++++
 .../components/Dialogs/BaseDialog.spec.js     | 21 +++++++
 .../DialogComponents/CourseNameDate.spec.js   | 31 ++++++++++
 .../UserSelectForCourse.spec.js               | 56 +++++++++++++++++++
 5 files changed, 129 insertions(+), 13 deletions(-)
 delete mode 100644 src/components/Dialogs/vue-component-boiler-plate.vue
 create mode 100644 tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
 create mode 100644 tests/unit/components/Dialogs/BaseDialog.spec.js
 create mode 100644 tests/unit/components/Dialogs/DialogComponents/CourseNameDate.spec.js
 create mode 100644 tests/unit/components/Dialogs/DialogComponents/UserSelectForCourse.spec.js

diff --git a/src/components/Dialogs/vue-component-boiler-plate.vue b/src/components/Dialogs/vue-component-boiler-plate.vue
deleted file mode 100644
index d127bf8..0000000
--- a/src/components/Dialogs/vue-component-boiler-plate.vue
+++ /dev/null
@@ -1,13 +0,0 @@
-<template></template>
-<script>
-export default {
-  name: "CourseNameDate",
-  components: {},
-  props: {},
-  data() {
-    return {};
-  },
-  created() {},
-};
-</script>
-<style></style>
diff --git a/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
new file mode 100644
index 0000000..528989f
--- /dev/null
+++ b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
@@ -0,0 +1,21 @@
+import { createLocalVue, shallowMount } from "@vue/test-utils";
+import Vuetify from "vuetify";
+import Vuex from "vuex";
+import AddStudentsToCourseDialog from "@/components/Dialogs/AddStudentsToCourseDialog";
+
+const localVue = createLocalVue();
+localVue.use(Vuex);
+
+const testUser = { name: "test" };
+const testCourse = { name: "test", capacity: 100 };
+
+describe("SelectAddStudentsToCourseDialog", () => {
+  // vuetify has to be mocked for the v-autocomplete to not crash
+  let vuetify;
+  beforeEach(() => {
+    vuetify = new Vuetify();
+  });
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+});
diff --git a/tests/unit/components/Dialogs/BaseDialog.spec.js b/tests/unit/components/Dialogs/BaseDialog.spec.js
new file mode 100644
index 0000000..ae92296
--- /dev/null
+++ b/tests/unit/components/Dialogs/BaseDialog.spec.js
@@ -0,0 +1,21 @@
+import { createLocalVue, shallowMount } from "@vue/test-utils";
+import Vuetify from "vuetify";
+import Vuex from "vuex";
+import BaseDialog from "@/components/Dialogs/BaseDialog";
+
+const localVue = createLocalVue();
+localVue.use(Vuex);
+
+const testUser = { name: "test" };
+const testCourse = { name: "test", capacity: 100 };
+
+describe("BaseDialog", () => {
+  // vuetify has to be mocked for the v-autocomplete to not crash
+  let vuetify;
+  beforeEach(() => {
+    vuetify = new Vuetify();
+  });
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+});
diff --git a/tests/unit/components/Dialogs/DialogComponents/CourseNameDate.spec.js b/tests/unit/components/Dialogs/DialogComponents/CourseNameDate.spec.js
new file mode 100644
index 0000000..c3568e7
--- /dev/null
+++ b/tests/unit/components/Dialogs/DialogComponents/CourseNameDate.spec.js
@@ -0,0 +1,31 @@
+import { createLocalVue, shallowMount } from "@vue/test-utils";
+import Vuetify from "vuetify";
+import Vuex from "vuex";
+import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
+
+const localVue = createLocalVue();
+localVue.use(Vuex);
+
+describe("CourseNameDate", () => {
+  // vuetify has to be mocked for the v-autocomplete to not crash
+  let vuetify;
+  beforeEach(() => {
+    vuetify = new Vuetify();
+  });
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+  it("course name and could be in html", async () => {
+    const wrapper = shallowMount(CourseNameDate, {
+      propsData: {
+        courseName: "testCourseName",
+        courseDate: "testCourseDate",
+      },
+      vuetify,
+    });
+
+    const html = wrapper.html();
+    expect(html.indexOf("testCourseName") !== -1).toBe(true);
+    expect(html.indexOf("testCourseDate") !== -1).toBe(true);
+  });
+});
diff --git a/tests/unit/components/Dialogs/DialogComponents/UserSelectForCourse.spec.js b/tests/unit/components/Dialogs/DialogComponents/UserSelectForCourse.spec.js
new file mode 100644
index 0000000..4156ae6
--- /dev/null
+++ b/tests/unit/components/Dialogs/DialogComponents/UserSelectForCourse.spec.js
@@ -0,0 +1,56 @@
+import { createLocalVue, shallowMount } from "@vue/test-utils";
+import Vuetify from "vuetify";
+import Vuex from "vuex";
+import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
+const localVue = createLocalVue();
+localVue.use(Vuex);
+
+const testUser = { name: "test" };
+
+describe("UserSelectForCourse", () => {
+  // vuetify has to be mocked for the v-autocomplete to not crash
+  let vuetify;
+  beforeEach(() => {
+    vuetify = new Vuetify();
+  });
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+  it("should have select user html", async () => {
+    const wrapper = shallowMount(UserSelectForCourse, {
+      propsData: {
+        availableSeatsInCourse: 100,
+        areNoAvailableSeats: false,
+        courseStartDate: "test",
+        updateCourseMembers: () => {},
+        isCourseSelected: true,
+        value: [],
+        selectedMembers: [testUser, testUser, testUser, testUser],
+      },
+      vuetify,
+    });
+
+    const html = wrapper.html();
+    const isFound = html.indexOf("userselect-stub") !== -1;
+    expect(isFound).toEqual(true);
+  });
+
+  it("should have select label with available seat reservations", async () => {
+    const wrapper = shallowMount(UserSelectForCourse, {
+      propsData: {
+        availableSeatsInCourse: 123456789,
+        areNoAvailableSeats: false,
+        courseStartDate: "test",
+        updateCourseMembers: () => {},
+        isCourseSelected: true,
+        value: [],
+        selectedMembers: [testUser, testUser, testUser, testUser],
+      },
+      vuetify,
+    });
+
+    const html = wrapper.html();
+    const isFound = html.indexOf("123456789") !== -1;
+    expect(isFound).toEqual(true);
+  });
+});
-- 
GitLab


From 019074a3887c3bc07538fc833b0fcf09baad2e1c Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Wed, 28 Apr 2021 16:01:34 -0600
Subject: [PATCH 29/63] unit tests

---
 .../components/Dialogs/BaseDialog.spec.js     | 45 +++++++++++++++++--
 1 file changed, 42 insertions(+), 3 deletions(-)

diff --git a/tests/unit/components/Dialogs/BaseDialog.spec.js b/tests/unit/components/Dialogs/BaseDialog.spec.js
index ae92296..dfd0cef 100644
--- a/tests/unit/components/Dialogs/BaseDialog.spec.js
+++ b/tests/unit/components/Dialogs/BaseDialog.spec.js
@@ -6,9 +6,6 @@ import BaseDialog from "@/components/Dialogs/BaseDialog";
 const localVue = createLocalVue();
 localVue.use(Vuex);
 
-const testUser = { name: "test" };
-const testCourse = { name: "test", capacity: 100 };
-
 describe("BaseDialog", () => {
   // vuetify has to be mocked for the v-autocomplete to not crash
   let vuetify;
@@ -18,4 +15,46 @@ describe("BaseDialog", () => {
   afterEach(() => {
     jest.clearAllMocks();
   });
+
+  it("should find dialog title in html", () => {
+    const wrapper = shallowMount(BaseDialog, {
+      propsData: {
+        dialogTitle: "123456789",
+      },
+      vuetify,
+    });
+
+    const html = wrapper.html();
+    console.log(html);
+    expect(html.indexOf("123456789") !== -1).toBe(true);
+  });
+
+  it("should find v-dialog in html", () => {
+    const wrapper = shallowMount(BaseDialog, {
+      propsData: {
+        dialogTitle: "123456789",
+      },
+      vuetify,
+    });
+
+    const html = wrapper.html();
+    console.log(html);
+    expect(html.indexOf("v-dialog-stub") !== -1).toBe(true);
+  });
+
+  it("should find yoda head in html", () => {
+    const wrapper = shallowMount(BaseDialog, {
+      propsData: {
+        dialogTitle: "123456789",
+      },
+      vuetify,
+    });
+
+    const html = wrapper.html();
+    expect(
+      html.indexOf(
+        `<div class="hide-overflow-container"><img src="@/assets/images/logos/YodaHead.png" alt="Yoda Head" class="yoda-head"></div>`
+      ) !== -1
+    ).toBe(true);
+  });
 });
-- 
GitLab


From 4dc7a4944c7899863093d6a48c3e25d95075b3b0 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 09:07:18 -0600
Subject: [PATCH 30/63] unit tests

---
 .../Dialogs/AddStudentsToCourseDialog.vue     | 63 ++++++++++---------
 .../Dialogs/AddStudentsToCourseDialog.spec.js | 49 ++++++++++++++-
 2 files changed, 82 insertions(+), 30 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index 0297a08..ed19ba1 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -1,15 +1,25 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
-    <v-card-text :style="{ height: '300px', display: 'flex', 'flex-direction': 'column', 'justify-content':'flex-end', 'padding-bottom':'150px' }">
+    <v-card-text
+      :style="{
+        height: '300px',
+        display: 'flex',
+        'flex-direction': 'column',
+        'justify-content': 'flex-end',
+        'padding-bottom': '150px',
+      }"
+    >
       <CourseNameDate
-          :course-name="course.name"
-          :course-date="course.startDate"
-       />
-      <div :style="{height:'10px'}"></div>
-      <UserSelectForCourse :are-no-available-seats="areNoAvailableSeats"
-      :available-seats-in-course="availableSeatsInCourse"
-      :update-course-members="updateCourseMembers"
-      :is-course-selected="true"  />
+        :course-name="course.name"
+        :course-date="course.startDate"
+      />
+      <div :style="{ height: '10px' }"></div>
+      <UserSelectForCourse
+        :are-no-available-seats="areNoAvailableSeats"
+        :available-seats-in-course="availableSeatsInCourse"
+        :update-course-members="updateCourseMembers"
+        :is-course-selected="true"
+      />
     </v-card-text>
 
     <template v-slot:actions>
@@ -44,20 +54,17 @@ import CourseService from "@/api/services/training";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
 
 export default {
-  name: "AddStudentDialog",
+  name: "AddStudentsToCourseDialog",
   components: { BaseDialog, UserSelectForCourse, CourseNameDate },
   props: {
     showDialog: Boolean,
-    courseId: Number
+    courseId: Number,
   },
   methods: {
     async addSelectedUsersToSelectedCourses() {
       this.isAddingBusy = true;
       try {
-        await CourseService.addStudents(
-          this.courseId,
-          this.courseMembers
-        );
+        await CourseService.addStudents(this.courseId, this.courseMembers);
       } catch (error) {
         console.error("!add courses error!", error);
         this.$store.commit(SET_ERROR_MESSAGE, error);
@@ -66,7 +73,7 @@ export default {
         this.$refs.courseSelect && this.$refs.courseSelect.clear();
         this.isAddingBusy = false;
         this.isAdding = false;
-        this.course = { name: "", startDate: "", };
+        this.course = { name: "", startDate: "" };
         this.updateCourseMembers([]);
         this.$emit("close-dialog");
       }
@@ -75,21 +82,21 @@ export default {
       this.courseMembers = members;
     },
     cancel() {
-      this.$emit('close-dialog');
-    }
+      this.$emit("close-dialog");
+    },
   },
   data() {
     return {
       state: {
-        isAddingBusy: false
+        isAddingBusy: false,
       },
       course: {
-        name: '',
-        startDate: '',
+        name: "",
+        startDate: "",
       },
       courseRegistrations: [],
       courseMembers: [],
-    }
+    };
   },
   computed: {
     areNoAvailableSeats() {
@@ -108,15 +115,15 @@ export default {
     },
     canSubmit() {
       return !this.areNoAvailableSeats && this.courseMembers.length > 0;
-    }
+    },
   },
   async created() {
     this.course = await CourseService.getCourse(this.courseId);
-    const result = await CourseService.getCourseRegistrationsById(this.course.id);
+    const result = await CourseService.getCourseRegistrationsById(
+      this.course.id
+    );
     this.courseRegistrations = result.registrations;
-  }
+  },
 };
 </script>
-<style>
-
-</style>
+<style></style>
diff --git a/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
index 528989f..aad8194 100644
--- a/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
@@ -6,16 +6,61 @@ import AddStudentsToCourseDialog from "@/components/Dialogs/AddStudentsToCourseD
 const localVue = createLocalVue();
 localVue.use(Vuex);
 
-const testUser = { name: "test" };
-const testCourse = { name: "test", capacity: 100 };
+const getShallowMount = (vuetify) => {
+  return shallowMount(AddStudentsToCourseDialog, {
+    propsData: {
+      showDialog: true,
+      courseId: 1,
+    },
+    mocks: {
+      $store: {
+        state: {
+          user: {
+            user: {
+              permission: "Admin",
+            },
+          },
+          tutorial: {
+            tooltipName: "test",
+          },
+          userPreferences: {
+            userPreference: {
+              darkMode: true,
+            },
+          },
+        },
+      },
+    },
+    vuetify,
+  });
+};
+
+const checkHtml = (html, checkText) => {
+  return html.indexOf(checkText) !== -1;
+};
+
+const checkWrapperForHtml = (wrapper, text) => {
+  const html = wrapper.html();
+  expect(checkHtml(html, text)).toEqual(true);
+};
 
 describe("SelectAddStudentsToCourseDialog", () => {
   // vuetify has to be mocked for the v-autocomplete to not crash
   let vuetify;
+  const wrapper = getShallowMount(vuetify);
   beforeEach(() => {
     vuetify = new Vuetify();
   });
   afterEach(() => {
     jest.clearAllMocks();
   });
+  it("should have basedialog", () => {
+    checkWrapperForHtml(wrapper, "basedialog-stub");
+  });
+  it("dialog title should be add student to course", () => {
+    checkWrapperForHtml(wrapper, `dialogtitle="Add Student to Course"`);
+  });
+  it("should have user select", () => {
+    checkWrapperForHtml(wrapper, `userselectforcourse-stub`);
+  });
 });
-- 
GitLab


From 24f8f3a24db92b52b1d0bdd980563d799ae3726a Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 09:08:28 -0600
Subject: [PATCH 31/63] unit tests

---
 tests/unit/components/Dialogs/BaseDialog.spec.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/tests/unit/components/Dialogs/BaseDialog.spec.js b/tests/unit/components/Dialogs/BaseDialog.spec.js
index dfd0cef..3258fe1 100644
--- a/tests/unit/components/Dialogs/BaseDialog.spec.js
+++ b/tests/unit/components/Dialogs/BaseDialog.spec.js
@@ -23,9 +23,7 @@ describe("BaseDialog", () => {
       },
       vuetify,
     });
-
     const html = wrapper.html();
-    console.log(html);
     expect(html.indexOf("123456789") !== -1).toBe(true);
   });
 
-- 
GitLab


From 29aec960005942c4b36e5aeb6cf194dd615bdeb5 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 09:42:44 -0600
Subject: [PATCH 32/63] unit tests

---
 .../Dialogs/AddStudentsToCourseDialog.spec.js | 46 +++++++++++++++++++
 1 file changed, 46 insertions(+)

diff --git a/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
index aad8194..8b17eee 100644
--- a/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
@@ -1,7 +1,9 @@
 import { createLocalVue, shallowMount } from "@vue/test-utils";
+import flushPromises from "flush-promises";
 import Vuetify from "vuetify";
 import Vuex from "vuex";
 import AddStudentsToCourseDialog from "@/components/Dialogs/AddStudentsToCourseDialog";
+import CourseService from "@/api/services/training";
 
 const localVue = createLocalVue();
 localVue.use(Vuex);
@@ -63,4 +65,48 @@ describe("SelectAddStudentsToCourseDialog", () => {
   it("should have user select", () => {
     checkWrapperForHtml(wrapper, `userselectforcourse-stub`);
   });
+  it("should call addStudents", async () => {
+    wrapper.vm.courseId = 1;
+    wrapper.vm.courseMembers = [{ id: 1 }];
+    CourseService.addStudents = jest.fn();
+    wrapper.vm.addSelectedUsersToSelectedCourses();
+    expect(CourseService.addStudents).toHaveBeenCalledTimes(1);
+  });
+  it("should call addStudents", async () => {
+    wrapper.vm.courseId = 1;
+    wrapper.vm.courseMembers = [{ id: 1 }];
+    CourseService.addStudents = jest.fn().mockResolvedValue({});
+    wrapper.vm.addSelectedUsersToSelectedCourses();
+    expect(CourseService.addStudents).toHaveBeenCalledTimes(1);
+  });
+  it("should handle error", async () => {
+    wrapper.vm.courseId = 1;
+    wrapper.vm.courseMembers = [{ id: 1 }];
+    wrapper.vm.$store.commit = jest.fn();
+    console.error = jest.fn();
+    CourseService.addStudents = jest.fn().mockRejectedValue({});
+    await wrapper.vm.addSelectedUsersToSelectedCourses();
+    expect(CourseService.addStudents).toHaveBeenCalledTimes(1);
+    expect(console.error).toHaveBeenCalled();
+    expect(wrapper.vm.$store.commit).toHaveBeenCalled();
+  });
+  it("cancel should emit close dialog", async () => {
+    wrapper.vm.$emit = jest.fn();
+    wrapper.vm.cancel();
+    expect(wrapper.vm.$emit).toHaveBeenCalledTimes(1);
+  });
+  it("can submit should return false if no available seats or no course members", async () => {
+    wrapper.vm.course.capacity = 0;
+    wrapper.vm.courseMembers = [];
+    wrapper.vm.courseRegistrations = [];
+    await wrapper.vm.$nextTick();
+    expect(wrapper.vm.canSubmit).toEqual(false);
+  });
+  it("can submit should return true if available seats or and course members", async () => {
+    wrapper.vm.course.capacity = 10;
+    wrapper.vm.courseMembers = [{ id: 1 }];
+    wrapper.vm.courseRegistrations = [];
+    await wrapper.vm.$nextTick();
+    expect(wrapper.vm.canSubmit).toEqual(true);
+  });
 });
-- 
GitLab


From 7181b0a3c77cb08271271ff383d9e99f56af412f Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 09:52:54 -0600
Subject: [PATCH 33/63] unit tests

---
 .../SelectAddStudentsToCourseDialog.spec.js   | 36 +++++++++++++++++--
 1 file changed, 33 insertions(+), 3 deletions(-)

diff --git a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
index cdc32f1..05af900 100644
--- a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
@@ -2,6 +2,7 @@ import { createLocalVue, shallowMount } from "@vue/test-utils";
 import Vuetify from "vuetify";
 import Vuex from "vuex";
 import SelectAddStudentsToCourseDialog from "@/components/Dialogs/SelectAddStudentsToCourseDialog";
+import CourseService from "@/api/services/training";
 
 const localVue = createLocalVue();
 localVue.use(Vuex);
@@ -32,7 +33,6 @@ describe("SelectAddStudentsToCourseDialog", () => {
     await wrapper.vm.$nextTick();
     expect(wrapper.props().selectedMembers).toHaveLength(4);
   });
-
   it("if areNoAvailableSeats is true then should show error on UserSelect", async () => {
     const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
@@ -47,7 +47,6 @@ describe("SelectAddStudentsToCourseDialog", () => {
     const userSelectWrapper = wrapper.find(".user-select-component");
     expect(userSelectWrapper.exists()).toBe(false);
   });
-
   it("courseMembers should start as empty array", async () => {
     const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
@@ -64,7 +63,6 @@ describe("SelectAddStudentsToCourseDialog", () => {
         wrapper.vm.courseMembers.length === 0
     ).toBe(true);
   });
-
   it("internalSelectedMembers should be equal to selectedMembers", async () => {
     const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
       propsData: {
@@ -80,4 +78,36 @@ describe("SelectAddStudentsToCourseDialog", () => {
       wrapper.vm.selectedMembers == wrapper.vm.internalSelectedMembers
     ).toBe(false);
   });
+  it("updateCourseMembers should call emit", async () => {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
+      propsData: {
+        value: [],
+        selectedMembers: [],
+      },
+      vuetify,
+    });
+    wrapper.vm.$emit = jest.fn();
+    wrapper.vm.updateCourseMembers([{ test: "test" }]);
+    expect(wrapper.vm.courseMembers).toEqual([{ test: "test" }]);
+    expect(wrapper.vm.$emit).toHaveBeenCalled();
+  });
+  it("updateCourseSelected should call getCourseRegistrationsById", async () => {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
+      propsData: {
+        value: [],
+        selectedMembers: [],
+      },
+      vuetify,
+    });
+    wrapper.vm.$emit = jest.fn();
+    CourseService.getCourseRegistrationsById = jest
+      .fn()
+      .mockResolvedValue({ registrations: [{ id: 1 }] });
+    await wrapper.vm.updateCourseSelected([{ id: 1, test: "test" }]);
+    expect(wrapper.vm.courseRegistrations).toEqual([{ id: 1 }]);
+    expect(wrapper.vm.$emit).toHaveBeenCalled();
+
+    await wrapper.vm.updateCourseSelected(false);
+    expect(wrapper.vm.course).toEqual({ name: "", startDate: "" });
+  });
 });
-- 
GitLab


From 7f947e2515332e491a146fb246bafd25dee0caa7 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 09:57:22 -0600
Subject: [PATCH 34/63] unit tests

---
 .../SelectAddStudentsToCourseDialog.spec.js   | 21 +++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
index 05af900..56b9292 100644
--- a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
@@ -110,4 +110,25 @@ describe("SelectAddStudentsToCourseDialog", () => {
     await wrapper.vm.updateCourseSelected(false);
     expect(wrapper.vm.course).toEqual({ name: "", startDate: "" });
   });
+  it("addSelectedUsersToSelectedCourses should call addStudentsToCourses", async () => {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
+      propsData: {
+        value: [],
+        selectedMembers: [],
+      },
+      vuetify,
+    });
+    wrapper.vm.$emit = jest.fn();
+    CourseService.addStudentsToCourses = jest.fn();
+    wrapper.vm.updateCourseMembers = jest.fn();
+    wrapper.vm.$refs.courseSelect.clear = jest.fn();
+    await wrapper.vm.addSelectedUsersToSelectedCourses();
+    expect(CourseService.addStudentsToCourses).toHaveBeenCalled();
+    expect(wrapper.vm.isAddingBusy).toEqual(false);
+    expect(wrapper.vm.isAdding).toEqual(false);
+    expect(wrapper.vm.course).toEqual({ name: "", startDate: "" });
+    expect(wrapper.vm.updateCourseMembers).toHaveBeenCalled();
+    expect(wrapper.vm.$emit).toHaveBeenCalled();
+    expect(wrapper.vm.$refs.courseSelect.clear).toHaveBeenCalled();
+  });
 });
-- 
GitLab


From e13b45049758210f8652daee452c3b316bc94c54 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 10:00:32 -0600
Subject: [PATCH 35/63] unit tests

---
 .../SelectAddStudentsToCourseDialog.spec.js   | 29 +++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
index 56b9292..25201ea 100644
--- a/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/SelectAddStudentsToCourseDialog.spec.js
@@ -131,4 +131,33 @@ describe("SelectAddStudentsToCourseDialog", () => {
     expect(wrapper.vm.$emit).toHaveBeenCalled();
     expect(wrapper.vm.$refs.courseSelect.clear).toHaveBeenCalled();
   });
+  it("addSelectedUsersToSelectedCourses should handle error call addStudentsToCourses ", async () => {
+    const wrapper = shallowMount(SelectAddStudentsToCourseDialog, {
+      propsData: {
+        value: [],
+        selectedMembers: [],
+      },
+      mocks: {
+        $store: {
+          commit: jest.fn(),
+        },
+      },
+      vuetify,
+    });
+    wrapper.vm.$emit = jest.fn();
+    console.error = jest.fn();
+    CourseService.addStudentsToCourses = jest.fn().mockRejectedValue();
+    wrapper.vm.updateCourseMembers = jest.fn();
+    wrapper.vm.$refs.courseSelect.clear = jest.fn();
+    await wrapper.vm.addSelectedUsersToSelectedCourses();
+    expect(console.error).toHaveBeenCalled();
+    expect(wrapper.vm.$store.commit).toHaveBeenCalled();
+    expect(CourseService.addStudentsToCourses).toHaveBeenCalled();
+    expect(wrapper.vm.isAddingBusy).toEqual(false);
+    expect(wrapper.vm.isAdding).toEqual(false);
+    expect(wrapper.vm.course).toEqual({ name: "", startDate: "" });
+    expect(wrapper.vm.updateCourseMembers).toHaveBeenCalled();
+    expect(wrapper.vm.$emit).toHaveBeenCalled();
+    expect(wrapper.vm.$refs.courseSelect.clear).toHaveBeenCalled();
+  });
 });
-- 
GitLab


From 17009d6dd397d4d5fcda29ad0c45ceedaed859fb Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 10:03:20 -0600
Subject: [PATCH 36/63] unit tests

---
 tests/unit/components/Dialogs/BaseDialog.spec.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/tests/unit/components/Dialogs/BaseDialog.spec.js b/tests/unit/components/Dialogs/BaseDialog.spec.js
index 3258fe1..219bdd7 100644
--- a/tests/unit/components/Dialogs/BaseDialog.spec.js
+++ b/tests/unit/components/Dialogs/BaseDialog.spec.js
@@ -34,9 +34,7 @@ describe("BaseDialog", () => {
       },
       vuetify,
     });
-
     const html = wrapper.html();
-    console.log(html);
     expect(html.indexOf("v-dialog-stub") !== -1).toBe(true);
   });
 
-- 
GitLab


From 597e768828e468f9ece5e2debc07bc24cecc7538 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 10:06:11 -0600
Subject: [PATCH 37/63] unit tests

---
 tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
index 8b17eee..c90698a 100644
--- a/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
+++ b/tests/unit/components/Dialogs/AddStudentsToCourseDialog.spec.js
@@ -1,5 +1,4 @@
 import { createLocalVue, shallowMount } from "@vue/test-utils";
-import flushPromises from "flush-promises";
 import Vuetify from "vuetify";
 import Vuex from "vuex";
 import AddStudentsToCourseDialog from "@/components/Dialogs/AddStudentsToCourseDialog";
-- 
GitLab


From 90781f4d3544b9528ff8f93789df2a330c828d1c Mon Sep 17 00:00:00 2001
From: "graham.smith" <graham.smith@bluechiptechnologies.net>
Date: Thu, 29 Apr 2021 12:44:59 -0400
Subject: [PATCH 38/63] change "class" to "course"

---
 src/components/CourseSelect.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/CourseSelect.vue b/src/components/CourseSelect.vue
index 784287b..19f372a 100644
--- a/src/components/CourseSelect.vue
+++ b/src/components/CourseSelect.vue
@@ -19,7 +19,7 @@
     hide-selected
     aria-required="true"
     @change="modelChanged"
-    placeholder="Class name"
+    placeholder="Course name"
     @click:prepend="setInputElementFocus($refs.courseAutocomplete)"
     class="course-select-component"
   >
-- 
GitLab


From 26604c2b7fa71a9eaec8f9824493ffea6b37c66e Mon Sep 17 00:00:00 2001
From: "graham.smith" <graham.smith@bluechiptechnologies.net>
Date: Thu, 29 Apr 2021 12:45:19 -0400
Subject: [PATCH 39/63] grammar fix

---
 .../Dialogs/DialogComponents/UserSelectForCourse.vue          | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index 509c5d4..fd89f2e 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -23,7 +23,9 @@
       "
       >You have
       {{ availableSeatsInCourse }}
-      available seat reservations</label
+      available seat reservation{{
+        availableSeatsInCourse === 1 ? "" : "s"
+      }}</label
     >
   </div>
 </template>
-- 
GitLab


From 7e4071e0fe496b0e800f348d833ab400a0472109 Mon Sep 17 00:00:00 2001
From: "graham.smith" <graham.smith@bluechiptechnologies.net>
Date: Thu, 29 Apr 2021 12:45:34 -0400
Subject: [PATCH 40/63] prettier

---
 src/components/Training/TrainingAttendance.vue            | 6 +++++-
 tests/unit/components/Training/TrainingAttendance.spec.js | 1 -
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/components/Training/TrainingAttendance.vue b/src/components/Training/TrainingAttendance.vue
index cd8e3ac..ebe52f1 100644
--- a/src/components/Training/TrainingAttendance.vue
+++ b/src/components/Training/TrainingAttendance.vue
@@ -114,7 +114,11 @@
         </v-btn>
       </div>
     </div>
-    <AddStudentsToCourseDialog :course-id="currentCourseId" :show-dialog="state.isAddingStudent" v-on:close-dialog="closeAddStudentDialog" />
+    <AddStudentsToCourseDialog
+      :course-id="currentCourseId"
+      :show-dialog="state.isAddingStudent"
+      v-on:close-dialog="closeAddStudentDialog"
+    />
   </div>
 </template>
 
diff --git a/tests/unit/components/Training/TrainingAttendance.spec.js b/tests/unit/components/Training/TrainingAttendance.spec.js
index cf23a76..2e48a5b 100644
--- a/tests/unit/components/Training/TrainingAttendance.spec.js
+++ b/tests/unit/components/Training/TrainingAttendance.spec.js
@@ -456,7 +456,6 @@ describe("TrainingAttendance", () => {
     expect(wrapper.vm.$store.commit).toHaveBeenCalledTimes(1);
   });
 
-
   it("should call fetchDebounced when options change", async () => {
     TrainingService.getCourseRegistrationsById = jest
       .fn()
-- 
GitLab


From 0adfaf85cd782c7946849aa3ec255dcbbd2ba83f Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 12:03:07 -0600
Subject: [PATCH 41/63] Refactor

---
 .../Dialogs/AddStudentsToCourseDialog.vue     | 25 +++++++++++--------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index ed19ba1..a33ff24 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -1,19 +1,11 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
-    <v-card-text
-      :style="{
-        height: '300px',
-        display: 'flex',
-        'flex-direction': 'column',
-        'justify-content': 'flex-end',
-        'padding-bottom': '150px',
-      }"
-    >
+    <v-card-text class="add-student-to-course-v-card">
       <CourseNameDate
         :course-name="course.name"
         :course-date="course.startDate"
       />
-      <div :style="{ height: '10px' }"></div>
+      <div class="vertical-padding"></div>
       <UserSelectForCourse
         :are-no-available-seats="areNoAvailableSeats"
         :available-seats-in-course="availableSeatsInCourse"
@@ -126,4 +118,15 @@ export default {
   },
 };
 </script>
-<style></style>
+<style>
+.add-student-to-course-v-card {
+  height: 300px;
+  display: flex;
+  flex-direction: column;
+  justify-content: flex-end;
+  padding-bottom: 150px;
+}
+.vertical-padding {
+  height: 10px;
+}
+</style>
-- 
GitLab


From 56622156b50a263eb37e0d0171021e390b2e4879 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 12:04:35 -0600
Subject: [PATCH 42/63] Refactor

---
 src/components/Dialogs/AddStudentsToCourseDialog.vue | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index a33ff24..2adb9d6 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -96,9 +96,6 @@ export default {
     },
     availableSeatsInCourse() {
       let seats = this.course.capacity;
-      if (isNaN(seats)) {
-        seats = 10;
-      }
       if (!isNaN(this.courseMembers.length)) {
         seats -= this.courseMembers.length;
       }
-- 
GitLab


From 3ffb90c0ddb1e4953ebabc8551d9f649f0824968 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 12:05:27 -0600
Subject: [PATCH 43/63] Refactor

---
 .../DialogComponents/UserSelectForCourse.vue    | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index 509c5d4..d2e4fa9 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -14,13 +14,6 @@
     <label
       v-if="isCourseSelected && !areNoAvailableSeats"
       class="text-left w-100 d-block text--white available-seats-label"
-      style="
-        width: 81%;
-        display: block;
-        position: relative;
-        left: 33px;
-        bottom: 11px;
-      "
       >You have
       {{ availableSeatsInCourse }}
       available seat reservations</label
@@ -68,4 +61,12 @@ export default {
   },
 };
 </script>
-<style></style>
+<style>
+.available-seats-label {
+  width: 81%;
+  display: block;
+  position: relative;
+  left: 33px;
+  bottom: 11px;
+}
+</style>
-- 
GitLab


From 342c3e0a9982600f0df2b40cce13f35a59f813d5 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 12:08:40 -0600
Subject: [PATCH 44/63] Refactor

---
 src/components/Dialogs/AddStudentsToCourseDialog.vue | 10 ++--------
 src/utils/course.js                                  |  8 ++++++++
 2 files changed, 10 insertions(+), 8 deletions(-)
 create mode 100644 src/utils/course.js

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index 2adb9d6..a6e302e 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -44,6 +44,7 @@ import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelec
 import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
 import CourseService from "@/api/services/training";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
+import { availableSeatsInCourse } from "@/utils/course";
 
 export default {
   name: "AddStudentsToCourseDialog",
@@ -94,14 +95,7 @@ export default {
     areNoAvailableSeats() {
       return this.availableSeatsInCourse < 0;
     },
-    availableSeatsInCourse() {
-      let seats = this.course.capacity;
-      if (!isNaN(this.courseMembers.length)) {
-        seats -= this.courseMembers.length;
-      }
-      seats -= this.courseRegistrations.length;
-      return seats;
-    },
+    availableSeatsInCourse,
     canSubmit() {
       return !this.areNoAvailableSeats && this.courseMembers.length > 0;
     },
diff --git a/src/utils/course.js b/src/utils/course.js
new file mode 100644
index 0000000..a58622b
--- /dev/null
+++ b/src/utils/course.js
@@ -0,0 +1,8 @@
+export const availableSeatsInCourse = () => {
+  let seats = this.course.capacity;
+  if (!isNaN(this.courseMembers.length)) {
+    seats -= this.courseMembers.length;
+  }
+  seats -= this.courseRegistrations.length;
+  return seats;
+};
-- 
GitLab


From 61deaaa7313fec3b091da818ed025000e7e8b901 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 12:09:28 -0600
Subject: [PATCH 45/63] Refactor

---
 .../Dialogs/SelectAddStudentsToCourseDialog.vue     | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index 6b2e138..da791bb 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -58,6 +58,7 @@ import { setInputElementFocus } from "@/utils/events";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
 import CourseService from "@/api/services/training";
 import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
+import { availableSeatsInCourse } from "@/utils/course";
 
 export default {
   name: "SelectAddStudentsToCourseDialog",
@@ -139,17 +140,7 @@ export default {
     areNoAvailableSeats() {
       return this.availableSeatsInCourse < 0;
     },
-    availableSeatsInCourse() {
-      let seats = this.course.capacity;
-      if (isNaN(seats)) {
-        seats = 10;
-      }
-      if (!isNaN(this.courseMembers.length)) {
-        seats -= this.courseMembers.length;
-      }
-      seats -= this.courseRegistrations.length;
-      return seats;
-    },
+    availableSeatsInCourse,
     canSubmit() {
       return (
         this.isCourseSelected &&
-- 
GitLab


From e8b581a8ed9d7639d9f5bf0b7bbfeac2fc8c2b22 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 13:06:20 -0600
Subject: [PATCH 46/63] Revert "Refactor"

This reverts commit 61deaaa7
---
 .../Dialogs/SelectAddStudentsToCourseDialog.vue     | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index da791bb..6b2e138 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -58,7 +58,6 @@ import { setInputElementFocus } from "@/utils/events";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
 import CourseService from "@/api/services/training";
 import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
-import { availableSeatsInCourse } from "@/utils/course";
 
 export default {
   name: "SelectAddStudentsToCourseDialog",
@@ -140,7 +139,17 @@ export default {
     areNoAvailableSeats() {
       return this.availableSeatsInCourse < 0;
     },
-    availableSeatsInCourse,
+    availableSeatsInCourse() {
+      let seats = this.course.capacity;
+      if (isNaN(seats)) {
+        seats = 10;
+      }
+      if (!isNaN(this.courseMembers.length)) {
+        seats -= this.courseMembers.length;
+      }
+      seats -= this.courseRegistrations.length;
+      return seats;
+    },
     canSubmit() {
       return (
         this.isCourseSelected &&
-- 
GitLab


From ad95097ffeae0fe879fd2404725f6c15b5746c47 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 13:06:25 -0600
Subject: [PATCH 47/63] Revert "Refactor"

This reverts commit 342c3e0a
---
 src/components/Dialogs/AddStudentsToCourseDialog.vue | 10 ++++++++--
 src/utils/course.js                                  |  8 --------
 2 files changed, 8 insertions(+), 10 deletions(-)
 delete mode 100644 src/utils/course.js

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index a6e302e..2adb9d6 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -44,7 +44,6 @@ import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelec
 import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
 import CourseService from "@/api/services/training";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
-import { availableSeatsInCourse } from "@/utils/course";
 
 export default {
   name: "AddStudentsToCourseDialog",
@@ -95,7 +94,14 @@ export default {
     areNoAvailableSeats() {
       return this.availableSeatsInCourse < 0;
     },
-    availableSeatsInCourse,
+    availableSeatsInCourse() {
+      let seats = this.course.capacity;
+      if (!isNaN(this.courseMembers.length)) {
+        seats -= this.courseMembers.length;
+      }
+      seats -= this.courseRegistrations.length;
+      return seats;
+    },
     canSubmit() {
       return !this.areNoAvailableSeats && this.courseMembers.length > 0;
     },
diff --git a/src/utils/course.js b/src/utils/course.js
deleted file mode 100644
index a58622b..0000000
--- a/src/utils/course.js
+++ /dev/null
@@ -1,8 +0,0 @@
-export const availableSeatsInCourse = () => {
-  let seats = this.course.capacity;
-  if (!isNaN(this.courseMembers.length)) {
-    seats -= this.courseMembers.length;
-  }
-  seats -= this.courseRegistrations.length;
-  return seats;
-};
-- 
GitLab


From 1194f3aa7544d0556fbe2ed93184e20973c4fa33 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 13:16:40 -0600
Subject: [PATCH 48/63] Refactor

---
 .../Dialogs/AddStudentsToCourseDialog.vue      | 12 ++++++------
 .../DialogComponents/CourseNameDate.vue        | 18 ++++++++++++++++--
 .../DialogComponents/UserSelectForCourse.vue   | 10 +++++-----
 3 files changed, 27 insertions(+), 13 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index 2adb9d6..9854d17 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -115,13 +115,13 @@ export default {
   },
 };
 </script>
-<style>
+<style scoped>
 .add-student-to-course-v-card {
-  height: 300px;
-  display: flex;
-  flex-direction: column;
-  justify-content: flex-end;
-  padding-bottom: 150px;
+  height: 300px !important;
+  display: flex !important;
+  flex-direction: column !important;
+  justify-content: flex-end !important;
+  padding-bottom: 150px !important;
 }
 .vertical-padding {
   height: 10px;
diff --git a/src/components/Dialogs/DialogComponents/CourseNameDate.vue b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
index 7262189..fb75c85 100644
--- a/src/components/Dialogs/DialogComponents/CourseNameDate.vue
+++ b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
@@ -23,7 +23,7 @@ export default {
   },
 };
 </script>
-<style scoped>
+<style scoped lang="scss">
 .course-name-date {
   display: flex;
   flex-direction: row;
@@ -34,11 +34,25 @@ export default {
 .course-name {
   font-size: 15px;
   font-weight: bold;
-  color: white;
 }
 
 .course-date {
   font-size: 15px;
   color: white;
 }
+
+.theme--light {
+  .course-name-date {
+    display: flex;
+    flex-direction: row;
+    justify-content: space-between;
+    padding: 0 3em;
+  }
+  .course-name {
+    color: black;
+  }
+  .course-date {
+    color: black;
+  }
+}
 </style>
diff --git a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
index d2e4fa9..195500c 100644
--- a/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
+++ b/src/components/Dialogs/DialogComponents/UserSelectForCourse.vue
@@ -63,10 +63,10 @@ export default {
 </script>
 <style>
 .available-seats-label {
-  width: 81%;
-  display: block;
-  position: relative;
-  left: 33px;
-  bottom: 11px;
+  width: 81% !important;
+  display: block !important;
+  position: relative !important;
+  left: 33px !important;
+  bottom: 11px !important;
 }
 </style>
-- 
GitLab


From 19696307b0d7ab9ec5c597dcf568272a9259e4ea Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 13:26:55 -0600
Subject: [PATCH 49/63] Refactor

---
 .../Dialogs/AddStudentsToCourseDialog.vue       | 12 ++++++------
 .../Dialogs/SelectAddStudentsToCourseDialog.vue | 15 ++++++---------
 src/utils/course.js                             | 17 +++++++++++++++++
 3 files changed, 29 insertions(+), 15 deletions(-)
 create mode 100644 src/utils/course.js

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index 9854d17..cbe1c2b 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -44,6 +44,7 @@ import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelec
 import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
 import CourseService from "@/api/services/training";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
+import * as CourseUtils from "@/utils/course";
 
 export default {
   name: "AddStudentsToCourseDialog",
@@ -95,12 +96,11 @@ export default {
       return this.availableSeatsInCourse < 0;
     },
     availableSeatsInCourse() {
-      let seats = this.course.capacity;
-      if (!isNaN(this.courseMembers.length)) {
-        seats -= this.courseMembers.length;
-      }
-      seats -= this.courseRegistrations.length;
-      return seats;
+      return CourseUtils.availableSeatsInCourse(
+        this.course.capacity,
+        this.courseMembers,
+        this.courseRegistrations
+      );
     },
     canSubmit() {
       return !this.areNoAvailableSeats && this.courseMembers.length > 0;
diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index 6b2e138..72e3c3f 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -58,6 +58,7 @@ import { setInputElementFocus } from "@/utils/events";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
 import CourseService from "@/api/services/training";
 import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
+import * as CourseUtils from "@/utils/course";
 
 export default {
   name: "SelectAddStudentsToCourseDialog",
@@ -140,15 +141,11 @@ export default {
       return this.availableSeatsInCourse < 0;
     },
     availableSeatsInCourse() {
-      let seats = this.course.capacity;
-      if (isNaN(seats)) {
-        seats = 10;
-      }
-      if (!isNaN(this.courseMembers.length)) {
-        seats -= this.courseMembers.length;
-      }
-      seats -= this.courseRegistrations.length;
-      return seats;
+      return CourseUtils.availableSeatsInCourse(
+        this.course.capacity,
+        this.courseMembers,
+        this.courseRegistrations
+      );
     },
     canSubmit() {
       return (
diff --git a/src/utils/course.js b/src/utils/course.js
new file mode 100644
index 0000000..38d95e1
--- /dev/null
+++ b/src/utils/course.js
@@ -0,0 +1,17 @@
+export const availableSeatsInCourse = (
+  capacity,
+  selectedMembers,
+  registrations
+) => {
+  if (!Number.isInteger(capacity)) {
+    console.warn("capacity was not an integer");
+    return 0;
+  }
+  if (Array.isArray(selectedMembers)) {
+    capacity -= selectedMembers.length;
+  }
+  if (Array.isArray(registrations)) {
+    capacity -= registrations.length;
+  }
+  return capacity;
+};
-- 
GitLab


From 3766e4e1f1cc8e43edee0474b771fa814ccf6940 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 13:27:20 -0600
Subject: [PATCH 50/63] Refactor

---
 src/components/CourseSelect.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/CourseSelect.vue b/src/components/CourseSelect.vue
index 784287b..19f372a 100644
--- a/src/components/CourseSelect.vue
+++ b/src/components/CourseSelect.vue
@@ -19,7 +19,7 @@
     hide-selected
     aria-required="true"
     @change="modelChanged"
-    placeholder="Class name"
+    placeholder="Course name"
     @click:prepend="setInputElementFocus($refs.courseAutocomplete)"
     class="course-select-component"
   >
-- 
GitLab


From c17156b9b3075084867425969382f2d1dd465ebd Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 14:13:49 -0600
Subject: [PATCH 51/63] Refactor

---
 .../Dialogs/AddStudentsToCourseDialog.vue     | 33 ++++++++-----
 .../SelectAddStudentsToCourseDialog.vue       | 37 ++++++++-------
 src/utils/course.js                           | 47 ++++++++++++++++++-
 3 files changed, 88 insertions(+), 29 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index cbe1c2b..38eff7c 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -7,8 +7,20 @@
       />
       <div class="vertical-padding"></div>
       <UserSelectForCourse
-        :are-no-available-seats="areNoAvailableSeats"
-        :available-seats-in-course="availableSeatsInCourse"
+        :are-no-available-seats="
+          availableSeatsInCourse(
+            this.course.capacity,
+            this.courseMembers,
+            this.courseRegistrations
+          ) < 0
+        "
+        :available-seats-in-course="
+          availableSeatsInCourse(
+            this.course.capacity,
+            this.courseMembers,
+            this.courseRegistrations
+          )
+        "
         :update-course-members="updateCourseMembers"
         :is-course-selected="true"
       />
@@ -44,7 +56,7 @@ import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelec
 import CourseNameDate from "@/components/Dialogs/DialogComponents/CourseNameDate";
 import CourseService from "@/api/services/training";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
-import * as CourseUtils from "@/utils/course";
+import { availableSeatsInCourse, canAddStudentsToCourse } from "@/utils/course";
 
 export default {
   name: "AddStudentsToCourseDialog",
@@ -54,6 +66,8 @@ export default {
     courseId: Number,
   },
   methods: {
+    availableSeatsInCourse,
+    canAddStudentsToCourse,
     async addSelectedUsersToSelectedCourses() {
       this.isAddingBusy = true;
       try {
@@ -92,19 +106,14 @@ export default {
     };
   },
   computed: {
-    areNoAvailableSeats() {
-      return this.availableSeatsInCourse < 0;
-    },
-    availableSeatsInCourse() {
-      return CourseUtils.availableSeatsInCourse(
+    canSubmit() {
+      return this.canAddStudentsToCourse(
+        true,
         this.course.capacity,
         this.courseMembers,
-        this.courseRegistrations
+        this.course.registrations
       );
     },
-    canSubmit() {
-      return !this.areNoAvailableSeats && this.courseMembers.length > 0;
-    },
   },
   async created() {
     this.course = await CourseService.getCourse(this.courseId);
diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index 72e3c3f..7da3931 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -16,8 +16,20 @@
       ></v-text-field>
       <div :style="{ height: '40px' }"></div>
       <UserSelectForCourse
-        :are-no-available-seats="areNoAvailableSeats"
-        :available-seats-in-course="availableSeatsInCourse"
+        :are-no-available-seats="
+          availableSeatsInCourse(
+            this.course.capacity,
+            this.courseMembers,
+            this.courseRegistrations
+          ) < 0
+        "
+        :available-seats-in-course="
+          availableSeatsInCourse(
+            this.course.capacity,
+            this.courseMembers,
+            this.courseRegistrations
+          )
+        "
         :course-start-date="course.startDate"
         :update-course-members="updateCourseMembers"
         :is-course-selected="isCourseSelected"
@@ -58,7 +70,7 @@ import { setInputElementFocus } from "@/utils/events";
 import { SET_ERROR_MESSAGE, SET_ERROR_DIALOG } from "@/store/mutation-types";
 import CourseService from "@/api/services/training";
 import UserSelectForCourse from "@/components/Dialogs/DialogComponents/UserSelectForCourse";
-import * as CourseUtils from "@/utils/course";
+import { availableSeatsInCourse, canAddStudentsToCourse } from "@/utils/course";
 
 export default {
   name: "SelectAddStudentsToCourseDialog",
@@ -88,6 +100,8 @@ export default {
     isAddingBusy: false,
   }),
   methods: {
+    canAddStudentsToCourse,
+    availableSeatsInCourse,
     updateCourseMembers(members) {
       this.courseMembers = members;
       this.$emit("update-selected-members", members);
@@ -137,21 +151,12 @@ export default {
     setInputElementFocus,
   },
   computed: {
-    areNoAvailableSeats() {
-      return this.availableSeatsInCourse < 0;
-    },
-    availableSeatsInCourse() {
-      return CourseUtils.availableSeatsInCourse(
+    canSubmit() {
+      return this.canAddStudentsToCourse(
+        this.isCourseSelected,
         this.course.capacity,
         this.courseMembers,
-        this.courseRegistrations
-      );
-    },
-    canSubmit() {
-      return (
-        this.isCourseSelected &&
-        !this.areNoAvailableSeats &&
-        this.courseMembers.length > 0
+        this.course.registrations
       );
     },
     isCourseSelected() {
diff --git a/src/utils/course.js b/src/utils/course.js
index 38d95e1..b40f448 100644
--- a/src/utils/course.js
+++ b/src/utils/course.js
@@ -4,7 +4,7 @@ export const availableSeatsInCourse = (
   registrations
 ) => {
   if (!Number.isInteger(capacity)) {
-    console.warn("capacity was not an integer");
+    console.warn("capacity was not an integer so default course capacity is 0");
     return 0;
   }
   if (Array.isArray(selectedMembers)) {
@@ -15,3 +15,48 @@ export const availableSeatsInCourse = (
   }
   return capacity;
 };
+
+export const canAddStudentsToCourse = (
+  isCourseSelected,
+  capacity,
+  selectedMembers,
+  registrations
+) => {
+  //default to allow
+  let canAddStudentsToCourse = true;
+
+  //check if course is selected
+  if (!isCourseSelected) {
+    console.warn("course is not selected and can't add students to course");
+    canAddStudentsToCourse = false;
+  }
+
+  //check if seats are available
+  const seatsAvailable = availableSeatsInCourse(
+    capacity,
+    selectedMembers,
+    registrations
+  );
+  if (seatsAvailable < 0) {
+    canAddStudentsToCourse = false;
+    console.warn(
+      "can't add students to course because seats are not available"
+    );
+  }
+
+  //check if selected members is an array
+  if (!Array.isArray(selectedMembers)) {
+    canAddStudentsToCourse = false;
+    console.warn(
+      "can't add students to course because selected members is not an array"
+    );
+  } else if (selectedMembers.length <= 0) {
+    canAddStudentsToCourse = false;
+    console.warn(
+      "can't add students to course because there are no selected members"
+    );
+  }
+
+  //return boolean
+  return canAddStudentsToCourse;
+};
-- 
GitLab


From 0b5f776e6be22d804023a37941e2a253f8132523 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 14:18:15 -0600
Subject: [PATCH 52/63] Refactor

---
 src/components/Dialogs/AddStudentsToCourseDialog.vue       | 1 +
 src/components/Dialogs/SelectAddStudentsToCourseDialog.vue | 1 +
 src/components/Training/TrainingAttendance.vue             | 1 +
 3 files changed, 3 insertions(+)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index 38eff7c..adc59da 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -72,6 +72,7 @@ export default {
       this.isAddingBusy = true;
       try {
         await CourseService.addStudents(this.courseId, this.courseMembers);
+        this.$emit("update-students");
       } catch (error) {
         console.error("!add courses error!", error);
         this.$store.commit(SET_ERROR_MESSAGE, error);
diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index 7da3931..ab9b254 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -126,6 +126,7 @@ export default {
           [this.course],
           this.selectedMembers
         );
+        this.$emit("update-students");
       } catch (error) {
         console.error("!add courses error!", error);
         this.$store.commit(SET_ERROR_MESSAGE, error);
diff --git a/src/components/Training/TrainingAttendance.vue b/src/components/Training/TrainingAttendance.vue
index ebe52f1..ef0143c 100644
--- a/src/components/Training/TrainingAttendance.vue
+++ b/src/components/Training/TrainingAttendance.vue
@@ -118,6 +118,7 @@
       :course-id="currentCourseId"
       :show-dialog="state.isAddingStudent"
       v-on:close-dialog="closeAddStudentDialog"
+      v-on:update-students="fetchDebounced"
     />
   </div>
 </template>
-- 
GitLab


From 2a03fee13ad9df322b90bfed05c53644d1a765ac Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 14:21:31 -0600
Subject: [PATCH 53/63] Refactor

---
 src/utils/course.js | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/src/utils/course.js b/src/utils/course.js
index b40f448..79f2b69 100644
--- a/src/utils/course.js
+++ b/src/utils/course.js
@@ -27,7 +27,6 @@ export const canAddStudentsToCourse = (
 
   //check if course is selected
   if (!isCourseSelected) {
-    console.warn("course is not selected and can't add students to course");
     canAddStudentsToCourse = false;
   }
 
@@ -39,22 +38,13 @@ export const canAddStudentsToCourse = (
   );
   if (seatsAvailable < 0) {
     canAddStudentsToCourse = false;
-    console.warn(
-      "can't add students to course because seats are not available"
-    );
   }
 
   //check if selected members is an array
   if (!Array.isArray(selectedMembers)) {
     canAddStudentsToCourse = false;
-    console.warn(
-      "can't add students to course because selected members is not an array"
-    );
   } else if (selectedMembers.length <= 0) {
     canAddStudentsToCourse = false;
-    console.warn(
-      "can't add students to course because there are no selected members"
-    );
   }
 
   //return boolean
-- 
GitLab


From 2c4bd992a8e5083fe06016e2b0c85fa32b6eead9 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Thu, 29 Apr 2021 15:39:07 -0600
Subject: [PATCH 54/63] Refactor

---
 src/components/AddTeam.vue                     | 18 ++++++++----------
 .../SelectAddStudentsToCourseDialog.vue        | 11 +++++++++--
 2 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/src/components/AddTeam.vue b/src/components/AddTeam.vue
index 63c172f..9f347db 100644
--- a/src/components/AddTeam.vue
+++ b/src/components/AddTeam.vue
@@ -3,16 +3,7 @@
     :show-dialog="showDialog"
     :dialog-title="editMode ? `Edit team` : `Add a new team`"
   >
-    <v-form
-      v-model="valid"
-      id="add-team"
-      :style="{
-        padding: '2em',
-        'margin-bottom': '2em',
-        paddingTop: '0',
-        height: '425px',
-      }"
-    >
+    <v-form v-model="valid" id="add-team" class="add-team-form">
       <v-text-field
         label="Team name"
         v-model="toAdd.name"
@@ -157,4 +148,11 @@ export default {
     margin-top: 8px;
   }
 }
+
+.add-team-form {
+  padding: 2em !important;
+  margin-bottom: 2em !important;
+  padding-top: 0 !important;
+  height: 425px !important;
+}
 </style>
diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index ab9b254..2ed280a 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -1,6 +1,6 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add To Course">
-    <v-card-text :style="{ height: '400px' }">
+    <v-card-text id="select-add-student-dialog-text">
       <CourseSelect
         v-on:input="updateCourseSelected"
         ref="courseSelect"
@@ -14,7 +14,7 @@
         hint="* Required"
         :persistent-hint="true"
       ></v-text-field>
-      <div :style="{ height: '40px' }"></div>
+      <div id="select-add-student-dialog-text-divider"></div>
       <UserSelectForCourse
         :are-no-available-seats="
           availableSeatsInCourse(
@@ -170,4 +170,11 @@ export default {
 .course-select-dialog-user-select {
   margin-top: 40px;
 }
+#select-add-student-dialog-text {
+  height: 400px !important;
+}
+
+#select-add-student-dialog-text-divider {
+  height: 40px !important;
+}
 </style>
-- 
GitLab


From f7bb535d61adaa16699270f1447adb2a1237d64b Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Mon, 3 May 2021 10:14:27 -0600
Subject: [PATCH 55/63] rename variable

---
 src/utils/course.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/utils/course.js b/src/utils/course.js
index 79f2b69..d7b9c6a 100644
--- a/src/utils/course.js
+++ b/src/utils/course.js
@@ -23,11 +23,11 @@ export const canAddStudentsToCourse = (
   registrations
 ) => {
   //default to allow
-  let canAddStudentsToCourse = true;
+  let canAddToCourse = true;
 
   //check if course is selected
   if (!isCourseSelected) {
-    canAddStudentsToCourse = false;
+    canAddToCourse = false;
   }
 
   //check if seats are available
@@ -37,16 +37,16 @@ export const canAddStudentsToCourse = (
     registrations
   );
   if (seatsAvailable < 0) {
-    canAddStudentsToCourse = false;
+    canAddToCourse = false;
   }
 
   //check if selected members is an array
   if (!Array.isArray(selectedMembers)) {
-    canAddStudentsToCourse = false;
+    canAddToCourse = false;
   } else if (selectedMembers.length <= 0) {
-    canAddStudentsToCourse = false;
+    canAddToCourse = false;
   }
 
   //return boolean
-  return canAddStudentsToCourse;
+  return canAddToCourse;
 };
-- 
GitLab


From 8861bdda90c8e7e2e82183021a3b277a14ecb647 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Mon, 3 May 2021 10:37:19 -0600
Subject: [PATCH 56/63] refactor if statement

---
 src/utils/course.js | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/utils/course.js b/src/utils/course.js
index d7b9c6a..e5a2ea7 100644
--- a/src/utils/course.js
+++ b/src/utils/course.js
@@ -41,9 +41,7 @@ export const canAddStudentsToCourse = (
   }
 
   //check if selected members is an array
-  if (!Array.isArray(selectedMembers)) {
-    canAddToCourse = false;
-  } else if (selectedMembers.length <= 0) {
+  if (!Array.isArray(selectedMembers) || selectedMembers.length <= 0) {
     canAddToCourse = false;
   }
 
-- 
GitLab


From f2f58a0004e9b48ebd7056afd873dcec1116e1e2 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 4 May 2021 12:09:36 -0600
Subject: [PATCH 57/63] auto height for dialogs

---
 src/components/AddTeam.vue            | 7 -------
 src/components/Dialogs/BaseDialog.vue | 3 +--
 src/scss/custom.scss                  | 6 ++++++
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/src/components/AddTeam.vue b/src/components/AddTeam.vue
index 9f347db..2ca0c5f 100644
--- a/src/components/AddTeam.vue
+++ b/src/components/AddTeam.vue
@@ -148,11 +148,4 @@ export default {
     margin-top: 8px;
   }
 }
-
-.add-team-form {
-  padding: 2em !important;
-  margin-bottom: 2em !important;
-  padding-top: 0 !important;
-  height: 425px !important;
-}
 </style>
diff --git a/src/components/Dialogs/BaseDialog.vue b/src/components/Dialogs/BaseDialog.vue
index f4bae4b..5fb2599 100644
--- a/src/components/Dialogs/BaseDialog.vue
+++ b/src/components/Dialogs/BaseDialog.vue
@@ -70,14 +70,13 @@ export default {
 
 .v-card__actions {
   bottom: 0;
-  position: absolute;
   width: 100%;
   text-align: center;
   display: flex;
   flex-direction: row;
   justify-content: center;
   align-items: center;
-  margin-bottom: 40px;
+  padding-bottom: 40px !important;
 }
 
 .v-card__actions .v-btn:last-child {
diff --git a/src/scss/custom.scss b/src/scss/custom.scss
index a3e657f..582eb41 100644
--- a/src/scss/custom.scss
+++ b/src/scss/custom.scss
@@ -699,3 +699,9 @@ body {
     word-break: normal;
   }
 }
+
+.v-dialog .v-form {
+  padding: 2em !important;
+  padding-top: 0 !important;
+  margin-bottom: 0 !important;
+}
-- 
GitLab


From fc794e94791bcba5b32d6fcdf07be04a2c9d53ad Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 4 May 2021 12:14:01 -0600
Subject: [PATCH 58/63] auto height for dialogs

---
 src/components/Dialogs/AddStudentsToCourseDialog.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index adc59da..eebd755 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -127,11 +127,11 @@ export default {
 </script>
 <style scoped>
 .add-student-to-course-v-card {
-  height: 300px !important;
   display: flex !important;
   flex-direction: column !important;
   justify-content: flex-end !important;
-  padding-bottom: 150px !important;
+  padding-bottom: 27px !important;
+  padding-top: 27px !important;
 }
 .vertical-padding {
   height: 10px;
-- 
GitLab


From 34367861bb857c0a7528aece601d9b7bd297d259 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 4 May 2021 12:17:25 -0600
Subject: [PATCH 59/63] refactor css

---
 src/components/Dialogs/AddStudentsToCourseDialog.vue | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index eebd755..6a8d7ad 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -1,6 +1,8 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
-    <v-card-text class="add-student-to-course-v-card">
+    <v-card-text
+      class="add-student-to-course-v-card d-flex flex-column justify-end"
+    >
       <CourseNameDate
         :course-name="course.name"
         :course-date="course.startDate"
@@ -127,9 +129,6 @@ export default {
 </script>
 <style scoped>
 .add-student-to-course-v-card {
-  display: flex !important;
-  flex-direction: column !important;
-  justify-content: flex-end !important;
   padding-bottom: 27px !important;
   padding-top: 27px !important;
 }
-- 
GitLab


From bea77b342be48e2239a3ebebfbee789f8bef1384 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 4 May 2021 12:19:04 -0600
Subject: [PATCH 60/63] refactor css

---
 .../Dialogs/AddStudentsToCourseDialog.vue         | 15 ++-------------
 1 file changed, 2 insertions(+), 13 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index 6a8d7ad..b9ec102 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -1,13 +1,11 @@
 <template>
   <BaseDialog :show-dialog="showDialog" dialog-title="Add Student to Course">
-    <v-card-text
-      class="add-student-to-course-v-card d-flex flex-column justify-end"
-    >
+    <v-card-text class="d-flex flex-column justify-end pt-7 pb-7">
       <CourseNameDate
         :course-name="course.name"
         :course-date="course.startDate"
       />
-      <div class="vertical-padding"></div>
+      <div class="mt-3"></div>
       <UserSelectForCourse
         :are-no-available-seats="
           availableSeatsInCourse(
@@ -127,12 +125,3 @@ export default {
   },
 };
 </script>
-<style scoped>
-.add-student-to-course-v-card {
-  padding-bottom: 27px !important;
-  padding-top: 27px !important;
-}
-.vertical-padding {
-  height: 10px;
-}
-</style>
-- 
GitLab


From 68575045f88f5cb59bbf5e6e678d62f52e5debcd Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 4 May 2021 12:21:43 -0600
Subject: [PATCH 61/63] refactor css

---
 .../Dialogs/DialogComponents/CourseNameDate.vue    | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

diff --git a/src/components/Dialogs/DialogComponents/CourseNameDate.vue b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
index fb75c85..95a57c9 100644
--- a/src/components/Dialogs/DialogComponents/CourseNameDate.vue
+++ b/src/components/Dialogs/DialogComponents/CourseNameDate.vue
@@ -1,6 +1,6 @@
 <template>
-  <div class="course-name-date">
-    <span class="course-name">{{ courseName }}</span
+  <div class="course-name-date d-flex flex-row justify-space-between">
+    <span class="course-name font-weight-bold">{{ courseName }}</span
     ><span class="course-date">{{ courseDate }}</span>
   </div>
 </template>
@@ -25,15 +25,11 @@ export default {
 </script>
 <style scoped lang="scss">
 .course-name-date {
-  display: flex;
-  flex-direction: row;
-  justify-content: space-between;
   padding: 0 3em;
 }
 
 .course-name {
   font-size: 15px;
-  font-weight: bold;
 }
 
 .course-date {
@@ -42,12 +38,6 @@ export default {
 }
 
 .theme--light {
-  .course-name-date {
-    display: flex;
-    flex-direction: row;
-    justify-content: space-between;
-    padding: 0 3em;
-  }
   .course-name {
     color: black;
   }
-- 
GitLab


From e8af3a00fb4b61b155186022e06c548fc44d1885 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 4 May 2021 12:23:07 -0600
Subject: [PATCH 62/63] refactor css

---
 src/components/Dialogs/AddStudentsToCourseDialog.vue | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/components/Dialogs/AddStudentsToCourseDialog.vue b/src/components/Dialogs/AddStudentsToCourseDialog.vue
index b9ec102..71f7be8 100644
--- a/src/components/Dialogs/AddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/AddStudentsToCourseDialog.vue
@@ -28,8 +28,7 @@
 
     <template v-slot:actions>
       <v-btn
-        color=""
-        class="primary"
+        color="secondary"
         :disabled="state.isAddingBusy"
         text
         @click="cancel()"
@@ -38,9 +37,7 @@
       </v-btn>
       <v-btn
         :disabled="!canSubmit"
-        :class="{ 'not-operable': !canSubmit }"
         color="primary"
-        class="limegreen"
         :loading="state.isAddingBusy"
         text
         @click="addSelectedUsersToSelectedCourses"
-- 
GitLab


From 1f552703a6009acbb702fc40b056bde908ae7814 Mon Sep 17 00:00:00 2001
From: Keith Becker <keith.becker@recro.com>
Date: Tue, 4 May 2021 12:25:24 -0600
Subject: [PATCH 63/63] refactor css

---
 .../Dialogs/SelectAddStudentsToCourseDialog.vue      | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
index 2ed280a..4e7bc12 100644
--- a/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
+++ b/src/components/Dialogs/SelectAddStudentsToCourseDialog.vue
@@ -38,22 +38,12 @@
     </v-card-text>
 
     <template v-slot:actions>
-      <v-btn
-        color=""
-        class="primary"
-        :disabled="isAddingBusy"
-        text
-        @click="cancel()"
-      >
+      <v-btn color="secondary" :disabled="isAddingBusy" text @click="cancel()">
         Cancel
       </v-btn>
       <v-btn
         :disabled="!canSubmit"
         color="primary"
-        class="limegreen"
-        :class="{
-          'not-operable': !canSubmit,
-        }"
         :loading="isAddingBusy"
         text
         @click="addSelectedUsersToSelectedCourses()"
-- 
GitLab