From 93a17d4ef55d056bb98637fadfff673c15e20c21 Mon Sep 17 00:00:00 2001
From: Julien Calixte <jcalixte@isep.fr>
Date: Fri, 23 Aug 2019 10:06:09 +0200
Subject: [PATCH] fix: possibility to create accounts without login

---
 src/components/AccountList.vue        |  4 ++--
 src/components/AppResume.vue          | 12 ++++++------
 src/views/Home.vue                    | 14 ++++++--------
 src/views/User.vue                    | 24 ++++++++++++------------
 src/views/accounts/AccountNew.vue     |  4 ++++
 src/views/accounts/AccountSetting.vue |  5 ++++-
 6 files changed, 34 insertions(+), 29 deletions(-)

diff --git a/src/components/AccountList.vue b/src/components/AccountList.vue
index 2bff1ed..e789874 100644
--- a/src/components/AccountList.vue
+++ b/src/components/AccountList.vue
@@ -17,9 +17,9 @@
       </div>
     </div>
     <div v-else-if="fetched">
-      <awe-icon icon="inbox" />Les comptes s'afficheront ici.
+      <awe-icon icon="inbox" />&nbsp;Les comptes créés s'afficheront ici.
     </div>
-    <fab-button :to="{ name: 'account-new' }" v-if="user" :margin="true">
+    <fab-button :to="{ name: 'account-new' }" :margin="true">
       <span slot="fulltext">créer un compte</span>
     </fab-button>
   </div>
diff --git a/src/components/AppResume.vue b/src/components/AppResume.vue
index 3e22261..d604c46 100644
--- a/src/components/AppResume.vue
+++ b/src/components/AppResume.vue
@@ -16,9 +16,9 @@
         <div class="main-icon">
           <awe-icon icon="users"></awe-icon>
         </div>
-        <p>Choisissez de rendre le compte privé ou public.</p>
-        <p>Lorsque le compte est privé, seules les personnes du compte accéderont aux dépenses.</p>
-        <p>Lorsque le compte est public, toutes les personnes accédant au lien pourront voir, modifier ou supprimer les dépenses.</p>
+        <p>Choisissez de rendre le compte privé ou public</p>
+        <p>Ainsi, seules les personnes du compte privé accéderont aux dépenses.</p>
+        <p>Et pour les comptes publics, toutes personnes accédant au lien pourront interagir directement sans même à avoir à se connecter !</p>
       </div>
       <!-- <div class="column is-one-third">
         <div class="main-icon">
@@ -37,7 +37,7 @@
         </div>
         <p>
           Vaquant vous permet d'enregistrer les dépenses dans la monnaie locale de vos vacances
-          et se charge de faire la conversion pour vous au jour le jour. 😙
+          et se charge de faire la conversion pour vous au jour le jour.
         </p>
       </div>
       <div class="column is-one-third">
@@ -46,7 +46,7 @@
         </div>
         <p>
           Vaquant se met à jour en direct. Vous recevez automatiquement les
-          nouveaux comptes et dépenses sans même vous en rendre compte. 🎉
+          nouveaux comptes et dépenses sans même vous en rendre compte.
         </p>
       </div>
       <div class="column is-one-third">
@@ -55,7 +55,7 @@
         </div>
         <p>
           Vaquant est simple, sécurisée, rapide et accessible depuis un simple lien internet&nbsp;!
-          Votre partenaire idéal pour vos voyages&nbsp;! ⛺
+          Votre partenaire idéal pour vos voyages&nbsp;!
         </p>
       </div>
       <div class="column is-one-third">
diff --git a/src/views/Home.vue b/src/views/Home.vue
index 2d74603..b438eca 100644
--- a/src/views/Home.vue
+++ b/src/views/Home.vue
@@ -7,17 +7,15 @@
     </div>
     <div v-else>
       <div>
-        <img class="logo" src="../assets/logo.svg" title="logo Vaquant">
-        <h2 class="subtitle is-2">Gérer vos budgets de groupe en toute simplicité&nbsp;!</h2>
+        <img class="logo" src="../assets/logo.svg" title="logo Vaquant" />
+        <h2 class="title is-2">Gérer vos budgets de groupe en toute simplicité&nbsp;!</h2>
       </div>
-      <br>
-      <router-link
-        class="button is-primary"
-        :to="{ name: 'user' }"
-        v-t="'user.loginsignup'"></router-link>
-      <div class="account-list-container">
+      <br />
+      <router-link class="button is-primary" :to="{ name: 'user' }" v-t="'user.loginsignup'"></router-link>
+      <div class="account-list-container box">
         <account-list />
       </div>
+      <h3 class="title is-3">Fonctionnalités</h3>
       <app-resume />
     </div>
   </div>
diff --git a/src/views/User.vue b/src/views/User.vue
index 1325b56..60e3977 100644
--- a/src/views/User.vue
+++ b/src/views/User.vue
@@ -1,14 +1,18 @@
 <template>
   <div class="user">
     <div v-if="user">
-      <h2 class="title is-2">
-        {{ user.userId }}
-      </h2>
-      <lang-changer />
-      <div class="buttons is-centered">
-        <button class="button is-warning" type="button" @click="logout" v-t="'user.logout'"></button>
-        <button class="button is-warning" type="button" @click="purge" v-t="'user.purge'"></button>
-        <confirm-button class="is-danger" @confirm="remove">{{ $t('user.delete') }}</confirm-button>
+      <h2 class="title is-2">{{ user.userId }}</h2>
+      <div class="columns is-centered">
+        <div class="column">
+          <lang-changer />
+        </div>
+        <div class="column">
+          <div class="buttons is-centered">
+            <button class="button is-warning" type="button" @click="logout" v-t="'user.logout'"></button>
+            <button class="button is-warning" type="button" @click="purge" v-t="'user.purge'"></button>
+            <confirm-button class="is-danger" @confirm="remove">{{ $t('user.delete') }}</confirm-button>
+          </div>
+        </div>
       </div>
       <hr />
       <account-list />
@@ -352,8 +356,4 @@ export default class User extends Vue {
 .user {
   padding: 15px 0;
 }
-
-.buttons {
-  margin-top: 15px;
-}
 </style>
diff --git a/src/views/accounts/AccountNew.vue b/src/views/accounts/AccountNew.vue
index 9b6c1e5..22f9358 100644
--- a/src/views/accounts/AccountNew.vue
+++ b/src/views/accounts/AccountNew.vue
@@ -132,6 +132,10 @@ export default class AccountNew extends Vue {
     const aliases: string[] = this.users.map((user: IUser) =>
       user.alias ? user.alias.toLowerCase() : ''
     )
+    if (aliases.length === 0) {
+      notif.alert('Au moins une personne doit être créée.')
+      return false
+    }
     if (aliases.length !== new Set(aliases).size) {
       notif.alert('Les alias doivent être uniques.')
       return false
diff --git a/src/views/accounts/AccountSetting.vue b/src/views/accounts/AccountSetting.vue
index 9db81cf..a39b7f8 100644
--- a/src/views/accounts/AccountSetting.vue
+++ b/src/views/accounts/AccountSetting.vue
@@ -46,7 +46,7 @@
           <div class="message-header">
             <p>Actions</p>
           </div>
-          <div class="buttons is-centered">
+          <div class="buttons action-buttons is-centered">
             <confirm-button v-if="account.archive" class="is-warning" @confirm="active">
               <span v-t="'account.open'"></span>
             </confirm-button>
@@ -171,4 +171,7 @@ export default class AccountSetting extends BaseAccount {
 .user-id {
   font-family: 'Cutive Mono', monospace;
 }
+.action-buttons {
+  margin: 15px;
+}
 </style>