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

Validation de la réorganisation du code en une seule arborecence de packages #32

Closed
phtanguy opened this issue Apr 4, 2014 · 4 comments
Assignees

Comments

@phtanguy
Copy link

phtanguy commented Apr 4, 2014

J'aimerais que tu me valides la réorganisation du code en une seule arborescence de packages qu'il faudrait je pense améliorer encore :

  • Tout est maintenant dans com.deev.interaction.uav3i mis à part le package com.deev.interaction.touch qui contient du code transverse à certains autres de tes projets.
  • Dans uav3i, j'ai déplacé :
    • le package util (log, etc.) : là, c'est pas trop ambigu...
    • le package veto qui contient tout ce qui a trait à la communication (y compris les classes DTO) et à l'IHM du serveur veto.
  • J'ai mis les classes pour la gestion de la carte OSM dans le package com.deev.interaction.uav3i.ui.maps bien qu'elles servent aussi à l'IHM du veto. Pas top...
  • Il ne serait pas inutile de donner un nom plus parlant au package com.deev.interaction.uav3i.ui (IHM de la table) mais je n'ai pas trop d'idées...

On ne va pas y passer " 3 mois non plus...

@legras
Copy link
Owner

legras commented Apr 7, 2014

Hello,

C’est pushé sur master ?

Mes suggestions dans le texte.

Le 4 avr. 2014 à 17:46, Philippe TANGUY notifications@github.com a écrit :

J'aimerais que tu me valides la réorganisation du code en une seule arborescence de packages qu'il faudrait je pense améliorer encore :

• Tout est maintenant dans com.deev.interaction.uav3i mis à part le package com.deev.interaction.touch qui contient du code transverse à certains autres de tes projets.
• Dans uav3i, j'ai déplacé :
• le package util (log, etc.) : là, c'est pas trop ambigu...
• le package veto qui contient tout ce qui a trait à la communication (y compris les classes DTO) et à l'IHM du serveur veto.
• J'ai mis les classes pour la gestion de la carte OSM dans le package com.deev.interaction.uav3i.ui.maps bien qu'elles servent aussi à l'IHM du veto. Pas top…

Directement dans com.deev.interaction.uav3i.maps ?

• Il ne serait pas inutile de donner un nom plus parlant au package com.deev.interaction.uav3i.ui (IHM de la table) mais je n'ai pas trop d'idées…

com.deev.interaction.uav3i.enduser ?

On ne va pas y passer " 3 mois non plus…

Certes non !

@phtanguy
Copy link
Author

phtanguy commented Apr 7, 2014

Comment tu fais pour reprendre un ancien message ?

Sinon :

  • C’est pushé sur master ?
    Non, uniquement sur IHM-VetoServer-dev. Si ça te convient, on l'intègre au prochain merge ?
  • Directement dans com.deev.interaction.uav3i.maps ?
    Oui...
  • com.deev.interaction.uav3i.enduser ?
    J'acquiesce (je viens de m'étonner en l'écrivant sans faute du premier coup !!!)
  • Certes non !
    On verra au fil de l'eau pour les besoins à venir...

@legras
Copy link
Owner

legras commented Apr 7, 2014

Le 7 avr. 2014 à 14:50, Philippe TANGUY notifications@github.com a écrit :

Comment tu fais pour reprendre un ancien message ?

Je réponds mar mail !

@phtanguy
Copy link
Author

phtanguy commented Apr 7, 2014

T'es un futé toi !

On 7 avril 2014 18:33:13 CEST, "François Legras" notifications@github.com wrote:

Le 7 avr. 2014 à 14:50, Philippe TANGUY notifications@github.com a
écrit :

Comment tu fais pour reprendre un ancien message ?

Je réponds mar mail !


Reply to this email directly or view it on GitHub:
#32 (comment)

Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté.

@legras legras closed this as completed May 28, 2014
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

2 participants