Closed Brawdunoir closed 2 years ago
Stoakes
commented
2 years ago L'erreur en CI me fait penser à une que j'ai vue lors de la mise à jour des tests d'intégrations cet été.
Les données de base étaient fausses et le champ dateSignature n'était pas renseigné sur la CC. Ce problème était censé être corrigé. Est-ce que la modification de la FA part bien de la version sur master ?
Stoakes
commented
2 years ago Peux tu stp supprimer tous les - des messages de commits ?
Tous les commit doivent commencer par un verbe à l'infinitif.
Brawdunoir
commented
2 years ago L'erreur en CI me fait penser à une que j'ai vue lors de la mise à jour des tests d'intégrations cet été.
Les données de base étaient fausses et le champ dateSignature n'était pas renseigné sur la CC. Ce problème était censé être corrigé. Est-ce que la modification de la FA part bien de la version sur master ?
Il me semble que l’erreur en CI provient de la FS, et oui le document est bien identique à celui sur le master. Je ne vois pas pourquoi il y a une erreur puisque le champs dateSignature n’est pas nul (il est bien renseigné lors du chargement des fausses données).
Note que bien que j’ai modifié la FA (car elle faisait référence à une nomFormel de Prospect qui n’existe pas), les tests en local ne passent pas non plus.
Brawdunoir
commented
2 years ago Normalement j’ai répondu à tous tes commentaires (pour ceux dont j’ai une réponse du moins, pour le reste j’ai appelé à l’aide Rémi ^^). Quand tout sera validé et que tu n’auras plus de commentaires ou de remarques je m’occuperai de squash tous les commits et d’appliquer le style lint ;)
Stoakes
commented
2 years ago docker/traefik/letsencrypt/acme.json casse la distribution de Jeyser.
git pull veut effacer le fichier deja existant, or ce fichier contient les certificats en cours.
Mais je vais m'occuper d'éditer ça. Mon objectif est de merger cette PR aujourd'hui
Améliorations : — Ajout de templates Twig pour un affichage uniformisé des informations concernant les études dans les différents onglets et dans Vue CA ; — Ajout du statut « Finie » pour une étude, qui désigne une étude terminée mais pas encore auditée (par la CNJE) et donc pas encore clôturée ; — Quelques fonctions d’autocomplétion lors de l’ajout de phases pour une étude ; — Quelques ajout de couverture i18n ici et là ; — Meilleure prédiction du mandat courant pour une étude ; — Déplacement des contacts clients dans un nouvel onglet au sein d’une étude afin d’améliorer leur management et quelques diverses améliorations ; — On ne peut auditer que les documents qui ont effectivement été créé au sein de l’étude (plutôt que d’avoir la liste complète à l’avance).
Mise à jour : Depuis l’année dernière la CNJE conseille de signer un nouveau type de document lors d’études récurrentes avec un même client : la CCA (Convention Cadre Agile). Avec ce document il est possible de signer plusieurs BDC (Bon de Commande) qui sont alors équivalent des CE (mais en plus light car la CCA factorise diverses informations qui ne sont pas répétées dans les BDC. Cette PR propose un ajout de ces deux documents et leur implantation dans le CRM.
— Ajout BDC et CCA ; — Une nouvelle étude peut désormais être liée à une CCA ajouté précédemment (et donc liée indirectement à un client) OU directement à un client. Dans le premier cas on pourra créer un BDC et dans le second une CE ; — Implantation à tous les niveaux de ces nouveaux documents : Upload de documents type dans le dashboard admin, mise à jour dans l’overview des études et dans les études elles-mêmes.
Changements mineurs : 1) J’ai enlevé la référence à Thread dans les documents (elle ne servait pas —et je ne vois pas pourquoi on pourrait faire un thread sur un document— et posait parfois des problèmes lors de migration de la BD… 2) Rémi a modifié les boutons lorsque l’on clique sur un intervenant ou un client. Ces derniers étant buggy ou ne fonctionnant pas parfois (à revérifier et à voir avec Rémi car c’était au début du stage et ma mémoire n’est pas très fraîche…) 3) Un petit refactor de l’onglet VuCA pour qu’il soit plus convivial (à mon sens…) 4) Ajout de l’audit d’une étude (idem ça a été fait par Rémi) la suite de ce petit commit sera amené dans une prochaine PR avec un onglet dédié aux audits.