pixelhumain / communecter

Manage cities as a connected citizen (email & postalCode), produce openCityData, manage organizations, projects, events openly , an open societal approach
https://www.communecter.org/
Apache License 2.0
60 stars 26 forks source link

Ajouter un contact / pod contact #1312

Open sylvainbarbot opened 7 years ago

sylvainbarbot commented 7 years ago

1/ Ajouter un contact. Label à modifier : 'à faciliter' capture d ecran 2017-02-14 a 18 07 26 2/ Ajouter un contact, valider => toastr de validation "contacts créés" mais la popup ne se ferme pas 3/ Ajouter ou Modifier un contact, Bouton Annuler => le contact est ajouté. 4/ Pod contact => traduire les tooltips des boutons 5/ Pas de controle sur le contact => rien n'est obligatoire ? Possibilité de créer des contacts vides capture d ecran 2017-02-14 a 18 13 10 6/ Si je sélectionne un contact qui est un citoyen existant dans la BD, je peux quand même modifier son adresse mail ? Ca ne récupère pas l'adresse mail/téléphone de l'utilisateur ? 7/ Lorsque je fais un delete d'un contact, éviter le relaod de la page pour ça. Un remove de la ligne ça suffirait ? 8/ Si j'ajoute 2 fois le même citoyen existant dans communecter => pas de message et mise à jour du contact existant.

RaphaelRIVIERE commented 7 years ago

1 ) c'est fais 2) c'est fais 3) Pas encore trouver la source du bug 4) Fais 5) Fais 6) L'email qui se trouve dans point Contact et celui du Citoyen et différents, parce que il peut avoir un mail différents , un mail professionnel pour l'élément associer 7) Un peu chiant a faire , car les contacts sont stocker dans un tableau est ont un index, quand on supprime ça modifie l'ordre de ces index. C'est le même problème avec les adresses secondaires 8) les point contacts est assez libre, du coup je sais pas si c'est vraiment pertinent de contrôler si on a déjà ajouter cette utilisateur dans cette liste. par contre , faire une vérification si le point contact n'existe pas déjà Exemple si on a mit deux fois { "role" : "Directeur", tel : "XXXXXXXXX" } peux être interéssant ? wdyt? @sylvainbarbot @oceatoon @Kgneo

oceatoon commented 7 years ago

8) si les gars dupliquent un point de contact , j'ai envie de dire que c'est pas de notre faute mais la leur

sylvainbarbot commented 7 years ago
  1. Ca veut dire que c'est peut être pas la meilleure manière de stocker l'info ? Un identifiant pour la ligne de contact plutot qu'un index éviterait ce type de pb.
  2. ok : laissons libre. le cas que tu décris @RaphaelRIVIERE n'est pas bloquant. Comme dit @oceatoon : c'est leur problème.

D'une manière globale, je trouve que la fonctionnalité contact devrait être liée à la fonctionnalité de "membre" qui existe déjà. Quelque chose pour aussi faciliter la vie de l'utilisateur. Par exemple : lorsque je déclare un nouveau contact je peux aller choisir parmi la communauté des membres de l'élément. Avec la popup des contacts de tango ça pourrait être rapide à faire.

RaphaelRIVIERE commented 7 years ago

Rapide , je sais pas.

pour le moment la structure ressemble a ça

{
   "contacts" : [
          "name" : "name",
          "id" : "id"
          "role" : "role",
          "email" : "email
          "tel" : [ "num" ]
   ]
}

Apres Pourquoi pas revoir la structure quand on identifie plusieurs fois le meme user . Mais est-ce que ça vaut le coup de faire ça pour la 0.17, autant le faire pour co2 ? wdyt ? @sylvainbarbot @oceatoon @Kgneo