Ajout Doctype Avenant - Githubissues

PitouGames commented 4 years ago

Cette PR contient de multiples corrections dont la principale est l'ajout de la possibilité d'upload le doctype Avenant et donc de le générer.

Autre modifications :

Ajout d'un bouton de suppression d'un avenant (fix #52).
Ajout de champs 'numero' pour les avenants. Il n'était pas logique que leur numéro affiché soit leur id (commun entre toutes les études) et que cette infomration ne soit pas disponnible pour la génération de document.
Prix du JEH par défaut passe à 400€.
Ajout d'ancres pour la navigation sur les études pour renvoyer l'utilisateur sur son ancien onglet (tab).
Mise à jour de code Twig déprécié.

Je pense qu'il manque envore le doctype AvMisson, on regardera sans doute pour l'ajouter dans le futur.

Stoakes commented 4 years ago

Impressionant. Merci pour cette contribution.

Par contre on a perdu Travis-CI, il faut que je regarde ça avant que l'on merge.

Stoakes commented 4 years ago

Normalement c'est réglé pour Travis. Est-ce que tu peux stp repusher sur la branche pour que ca retrigger le build stp ?

PitouGames commented 4 years ago

Le test de Codacy est un peu chiant : pour moi ce ne sont pas des vrais problèmes qu'il rapporte. De plus il est passé sans problème sur https://github.com/n7consulting/Incipio/commit/75b8bac0739c237b7b67b377f5b1d20a026e0716 et je n'ai rien changé au code depuis, son comportement n'est donc pas toujours le même... Que dois-je faire ?

Stoakes commented 4 years ago

Pour codacy, on va ignorer ca. Je fait la review asap et on merge ca ensuite.

Stoakes commented 4 years ago

Pas la peine de trop trainer sur cette PR, donc je merge, quitte a repasser plus tard sur les onglets.

Merci @PitouGames

n7consulting / Incipio