search

OCSInventory-NG / AndroidAgent

OCS Inventory NG Android Agent
http://www.ocsinventory-ng.org/en/
GNU General Public License v2.0
28 stars 23 forks source link

Mise à jour du projet #8

Closed AnaelMobilia closed 7 years ago

AnaelMobilia commented 7 years ago

[Résumé] Fix de bugs (ouverts dans le dépôt OCS) : #9 #10 #4 + non remontée des cameras SDK 19 => 22 Recodage de l'affichage de l'inventaire Fix d'erreurs lint & codacy dans le projet Nettoyage du code & Suppression de code mort

gillesdubois commented 7 years ago

Bonjour,

Tout d'abord merci pour ta contribution.

J'ai testé ton nouvel agent tout build normalement et les test passent bien. Néanmoins il y a un soucis sur l'icône de l'agent, c'est l'ancienne qui s'affiche. Pourrait-tu remettre celle d'origine ou alors je peux t'envoyer la nouvelle ?

screenshot_1474979809

Merci d'avance, Gilles Dubois.

AnaelMobilia commented 7 years ago

@gillesdubois done :-)

gillesdubois commented 7 years ago

Salut,

Un petit soucis a été détecté, une perte de données a la remonté est constatée entre la v 2.1.5 et la PR que tu as envoyé.

Le test a été réalise sur 4 devices différents ( 2 Android 5.1 / 2 Android 6.0.X )

Ci-joint un example (un seul et unique téléphone) :

Avec l'ancien agent :

old-agent

Avec le nouvelle agent :

new-agent

Le problème pourrait-il venir de ta pull request ? Je commence l'investigation de mon côté :)

Si jamais ce n'est pas lié a ta PR, le reste est parfait, j'accepterai le merge et on corrigera par la suite.

Merci pour ta contribution.

Cordialement, Gilles Dubois.

AnaelMobilia commented 7 years ago

@gillesdubois ça m'évoque quelque chose, je pense que c'est lié à la PR. Je regarde :)

La plate-forme de tests est-elle accessible publiquement ?

AnaelMobilia commented 7 years ago

@gillesdubois imho lié à https://github.com/OCSInventory-NG/AndroidAgent/commit/a7835c3725aef1f451b806845d79495715859e9b#diff-eda82cb3261e2d07c81e44b5550f2292R69

Je regarde pour réactiver :)

AnaelMobilia commented 7 years ago

@gillesdubois Pourras-tu tester s'il te plaît ? Les caméras de l'émulateur remontent correctement :)

Je ne sais pas pourquoi @cdpointpoint avait désactivé cette partie du code ? (le commentaire du commit n'est pas en phase). Je pense que c'était un test ou alors un problème de crash de l'appli ?

gillesdubois commented 7 years ago

Après les derniers commits les Front et Back camera sont de retour ça venait bien du bout de code commenté :)

Il reste toujours quelques sections (moniteurs par exemples) non remontés mais ça ne vient pas de la PR, on fera ça sur une correction ultérieure.

La PR en elle même est OK, encore bravo pour le boulot réalisé, j'attends ton feu vert pour la valider ( Au cas ou tu aurais d'autre commit en attente )

Gilles Dubois.

AnaelMobilia commented 7 years ago

@gillesdubois peux-tu, s'il te plaît, créer des issues pour les problèmes rencontrés ? Ça me permettra d'avoir du grain à moudre :)

Je pense qu'on peut déjà intégrer la PR pour fermer un certain nombre d'issues.

A voir comment on fera pour la suite (Pull Request à gogo ou création d'une branche sur le dépôt parent avec intégration de mon compte dans les authorized ?)

Merci :)