From bee0eba7320f4988277208d8e47f7e30a7d88523 Mon Sep 17 00:00:00 2001
From: Dave <d.garnar@uk.cdw.com>
Date: Mon, 25 Jan 2021 15:48:46 +0000
Subject: [PATCH] moving otherTravelers in the role modal from a computed
 function to spreading in mapstate for clarity and code style matching

---
 src/components/modals/RoleModal.vue | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/components/modals/RoleModal.vue b/src/components/modals/RoleModal.vue
index f177455..619c145 100644
--- a/src/components/modals/RoleModal.vue
+++ b/src/components/modals/RoleModal.vue
@@ -20,7 +20,7 @@
     </ul>
     <ul class="tokens" v-if="tab === 'otherTravelers'">
       <li
-        v-for="role in otherTravelers"
+        v-for="role in otherTravelers.values()"
         :class="[role.team]"
         :key="role.id"
         @click="setRole(role)"
@@ -54,9 +54,6 @@ export default {
   components: { Token, Modal },
   props: ["playerIndex"],
   computed: {
-    otherTravelers() {
-      return [...this.$store.state.otherTravelers.values()];
-    },
     availableRoles() {
       const availableRoles = [];
       const players = this.$store.state.players.players;
@@ -74,7 +71,8 @@ export default {
       return availableRoles;
     },
     ...mapState(["modals", "roles", "session"]),
-    ...mapState("players", ["players"])
+    ...mapState("players", ["players"]),
+    ...mapState(["otherTravelers"])
   },
   data() {
     return {