Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stabilité de la version 0.84 #3

Open
Pieurre opened this issue Mar 6, 2014 · 20 comments
Open

Stabilité de la version 0.84 #3

Pieurre opened this issue Mar 6, 2014 · 20 comments

Comments

@Pieurre
Copy link

Pieurre commented Mar 6, 2014

ping @ivamp
Bonjour,
Bonjour, j'aurais aimé savoir le plugin était bien stable stable en 0.84. , et si vous aviez des retours sur son installation en production dans cette version?

Merci beaucoup.

@ivamp
Copy link
Owner

ivamp commented Mar 6, 2014

J'ai modifié le plugin de sorte à faire les fonctions de base, pour moi j'ai aucun soucis en prod ;)
Pas d'installation spécifique il faut faire comme tous les plugins ;)

@Pieurre
Copy link
Author

Pieurre commented Mar 6, 2014

Merci beaucoup pour cette réponse ;)
J'ai quand même un petit Bug.
Je l'ai installé sur unne 0.84.5 et sur 0.84.4 et je constate juste que lors du choix d'une carte SIM, dans le bas le titre "Éléments associés" n'apparaît pas et le bouton de cet espace est fonctionnel mais n'est pas nommé "Ajouter".

Merci

@Pieurre
Copy link
Author

Pieurre commented Mar 6, 2014

Voici la différence entre 0.83 et 0.84 en image:
0.83
sim-0 83

0.84
sim-0 84

@ivamp
Copy link
Owner

ivamp commented Mar 6, 2014

Je check ca ;)

@ivamp
Copy link
Owner

ivamp commented Mar 6, 2014

Effectivement je ne l'avais pas vue ;)
Je regarde à ça

@ivamp
Copy link
Owner

ivamp commented Mar 6, 2014

Mais à part l'affichage ça fonctionne?

@Pieurre
Copy link
Author

Pieurre commented Mar 6, 2014

A priori oui , j'ai récupérer une base de 0.83 ca à l'air de fonctionner.

@ivamp
Copy link
Owner

ivamp commented Mar 6, 2014

Ok super, de mon côté n'étant pas devs à la base j'ai fait ça tout seul^^ je vais voir avec de vrai devs pour corriger le problème au plus vite ;)

@dethegeek
Copy link
Contributor

Bonjour

Une partie des bugs d'affichage sont résolus par ma contribution proposée pour fusion ce soir.

@Rople : ce serait utile de connaitre l'URL d'où proviennent les captures d'écran : si ma contribution ne résout pas ce problème je pourrai facilement cibler et corriger cela.

@Pieurre
Copy link
Author

Pieurre commented Mar 7, 2014

@dethegeek
Salut , merci pour ta contrib.
Les screens viennent d'une machine sur mon poste pour test avant mis en production je ne peux donc pas te donner accès.
Par contre c'est très facilement reproductible avec une fresh install de 0.84.5, ce qui est mon cas.

Merci.

@Pieurre
Copy link
Author

Pieurre commented Mar 7, 2014

@dethegeek @ivamp
Ce fonctionne maintenant je viens de faire le test et les champs sont maintenant ok.
@dethegeek : Comment se passe le "push" sur la forge ?

Merci beaucoup

@dethegeek
Copy link
Contributor

Bonjour

Tu as obtenu des accès pour verser tes changements sur la forge ?

Si oui, je ne sais pas ce que tu as mis en place pour ton environnement de
développement. Eventuellement on peut produire un diff pour que tu puisse
patcher ta copie à pousser dans le SVN.

Le 7 mars 2014 10:03, ropie [email protected] a écrit :

@dethegeek https://github.com/dethegeek @ivamphttps://github.com/ivamp
Ce fonctionne maintenant je viens de faire le test et les champs sont
maintenant ok.
@dethegeek https://github.com/dethegeek : Comment se passe le "push"
sur la forge ?

Merci beaucoup

Reply to this email directly or view it on GitHubhttps://github.com//issues/3#issuecomment-36978818
.

@ivamp
Copy link
Owner

ivamp commented Mar 7, 2014

Je n'ai pas accès à la forge il faut envoyé un mail directement à l'admin de la forge GLPI.
En tout cas je te remercie énormément pour la contribution je ne trouvais pas le moyen de fixer le problème. J'envoi un mail à l'admin en lui demandant si il peut rajouter ton nom ;)

@dethegeek
Copy link
Contributor

D'accord. Le plus important étant d'avoir un accès au dépôt pour le mettre
à jour... D'ailleurs il faut que je fasse la même chose pour le plugin que
j'ai repris.

Il reste du travail sur la compatibilité DataInjection pour nos 2 plugins :)

Le 7 mars 2014 15:04, ivamp [email protected] a écrit :

Je n'ai pas accès à la forge il faut envoyé un mail directement à l'admin
de la forge GLPI.
En tout cas je te remercie énormément pour la contribution je ne trouvais
pas le moyen de fixer le problème. J'envoi un mail à l'admin en lui
demandant si il peut rajouter ton nom ;)

Reply to this email directly or view it on GitHubhttps://github.com//issues/3#issuecomment-37026348
.

@dethegeek
Copy link
Contributor

@ropie : je suis passé en production la semaine dernière avec ce plugin. J'ai apporté pas mal de modifications pour corriger des choses qui manquaient pour moi, et (ré)activer des fonctionnalités.

Je pense avoir fait le tour du plugin et presque tout corrigé (j'ai encore un petit point à vérifier mais rien de méchant).

si tu veux tester le plugin dans un envronnement de test et remonter tes commentaires cela permettrait de fermer ce ticket :)

@Pieurre
Copy link
Author

Pieurre commented Mar 24, 2014

@dethegeek :
Salut,
Je passe le plugin demain. Ou puis-je télécharger la dernière version avec tes toutes dernières modifications?
Super boulot en tout cas ;)

++

@dethegeek
Copy link
Contributor

Salut : ivamp a mergé les dernières modifications ici. Vérifie que tu visionnes la branche master, et télécharge le zip, comme tu l'as déjà fait lors de ton premier téléchargement.

@Pieurre
Copy link
Author

Pieurre commented Mar 24, 2014

@dethegeek Ok super merci

@ivamp
Copy link
Owner

ivamp commented Mar 24, 2014

@dethegeek Voila j'ai eu des nouvelles pour la forge aurait tu une adresse mail à me communiquer à [email protected], tu sera également cité comme développeur ;)

@dethegeek
Copy link
Contributor

@ropie : j'ai oublié de préciser : le merge fait évoluer la version du plugin (la structure des tables a évolué), to dois donc passer par une phase de mise à jour, comme pour tout autre plugin. J'attends ton retour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants