From ecba587253821677374bad72408d8c51ba8558ce Mon Sep 17 00:00:00 2001
From: Stefan Zermatten <stefanzermatten@gmail.com>
Date: Tue, 16 Jun 2020 12:35:50 +0200
Subject: [PATCH] Fixed a bug with proficiency forms not editing proficiency
 correctly

---
 app/imports/api/properties/Effects.js               | 1 -
 app/imports/api/properties/Proficiencies.js         | 1 -
 app/imports/ui/properties/forms/ProficiencyForm.vue | 6 ++++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/imports/api/properties/Effects.js b/app/imports/api/properties/Effects.js
index 7ebc7ab7..9b5e98f4 100644
--- a/app/imports/api/properties/Effects.js
+++ b/app/imports/api/properties/Effects.js
@@ -38,7 +38,6 @@ let EffectSchema = new SimpleSchema({
 	},
 	'stats.$': {
 		type: String,
-		optional: true,
 	},
 });
 
diff --git a/app/imports/api/properties/Proficiencies.js b/app/imports/api/properties/Proficiencies.js
index 9d82d4fb..02a8f0ed 100644
--- a/app/imports/api/properties/Proficiencies.js
+++ b/app/imports/api/properties/Proficiencies.js
@@ -12,7 +12,6 @@ let ProficiencySchema = new SimpleSchema({
 	},
 	'stats.$': {
 		type: String,
-		optional: true,
 	},
 	// A number representing how proficient the character is
 	value: {
diff --git a/app/imports/ui/properties/forms/ProficiencyForm.vue b/app/imports/ui/properties/forms/ProficiencyForm.vue
index cd3c9296..e4bd20e4 100644
--- a/app/imports/ui/properties/forms/ProficiencyForm.vue
+++ b/app/imports/ui/properties/forms/ProficiencyForm.vue
@@ -9,9 +9,11 @@
     />
     <div class="layout row wrap justify-start proficiency-form">
       <smart-combobox
-        label="Skill"
+        label="Skills"
         class="mr-2"
         multiple
+        chips
+        deletable-chips
         :value="model.stats"
         :items="skillList"
         :error-messages="errors.stats"
@@ -22,7 +24,7 @@
         style="flex-basis: 300px;"
         :clearable="false"
         :value="model.value"
-        @change="change('stats', ...arguments)"
+        @change="change('value', ...arguments)"
       />
     </div>
   </div>