marcellinodour / MIDO-SVG

Other
0 stars 0 forks source link

Dev svg it3 : ajout méthode controlSettings, calcul x et y #26

Closed brulej closed 4 years ago

brulej commented 4 years ago

Bonjour,

Voici les modifications opérées par l'équipe SVG dans le cadre de cette pull request:

L'équipe SM/PO

marcellinodour commented 4 years ago

Check SMPO Done Note : les tests SVG ont été mis temporairement en commentaire car nous constatons un écart de calcul entre l'environnement eclipse et le WF.

Raphda commented 4 years ago

En effet, la position ne changera pas d'un ordinateur à un autre. Cependant, si nous fixons les positions en pixels en dur, dès que les données dans la BDD changeront (un nouveau cours, une formation supprimée etc) tous nos tests seront faux. Et si nous calculons les positions en pixels, nous referons le même calcul que dans la méthode defineObjectPosition et les tests seront validés dans tous les cas, ce qui n'a pas beaucoup d'intérêt. C'est pourquoi nous ne voyons pas comment faire nos tests unitaires. Qu'en pensez-vous ?

Raphda commented 4 years ago

Pour les changements déjà acceptés, il s'agit d'un bug de GitHub. Nous avons exécuté votre méthode (changer la branche master par une autre puis la remettre) mais cela n'a toujours pas prit en compte le dernier commit de la branche master.

brulej commented 4 years ago

Il me semble que les instructions projet n’ont pas été suivies. Voudriez-vous les relire et vérifier ?

.En demandant cette derniere request l'équipe SVG souhaitait attirer votre attention dans l'objectif de confirmer, la réévaluation de leur Itération. En effet, lors de notre dernière réunion teams, ces derniers ce sont rendus compte que la majeure partie de leur de travail n'avait pas été prise en compte (la classe Responsive-SVG).

Cela vient d'un bug github concernant l'actualisation de la branch de référence. Vous n'avez pas pu voir l'étendue de leur à travail à cause du nombre trop important de fichiers .

brulej commented 4 years ago

l'it3 est cloturée