search

KiwiHC16 / Abeille

Abeille pour Jeedom (Gateway ZiGate)
GNU Affero General Public License v3.0
60 stars 52 forks source link

GIT issue sur TRADFRISignalRepeater.json #1034

Closed tcharp38 closed 4 years ago

tcharp38 commented 4 years ago

Salut Kiwi Je galere a me realigner sur ton dernier master. Je suis dans un repo dans lequel je n'ai RIEN modifié donc je m'attends à aucun conflit. Neanmoins à l'issu du "git pull" ou meme "git pull --rebase" j'ai un conflit qui me bloque.

image

Ca te parle ? J'ai demandé à des copains pourtant maitrisant bien GIT et ils sechent aussi.

tcharp38 commented 4 years ago

Une autre info image

tcharp38 commented 4 years ago

Ok, voila une piste.

image

Ma conclusion est que tu as poussé les 2 fichiers avec juste la casse d'une lettre de difference. Manque de pot je suis dans un environnement Windows avec git bash, donc de mon côté pas de différence minuscule/majuscule, d'ou le clash insoluble.

Peux tu virer les 2 fichiers qui ne servent plus parmis les 4 suivants ?

Merci

KiwiHC16 commented 4 years ago

Oui je peux les virer mais alors on aura un autre soucis. J'ai ajouté ces fichier avec cette "casse" car un utilisateur a remonté l'info qu'il avait un repeter Ikea avec ce nom. Je ne sais pas trop comment on peut faire dans cette situation. Je vais regarder quel nom remonte avec celui que j'ai.

KiwiHC16 commented 4 years ago

Premier: Capture d’écran 2020-03-19 à 08 48 24

tcharp38 commented 4 years ago

Est ce que tout simplement il ne faut pas ignorer la casse de ce qui est remonté ? Je ne peux pas imaginer qu'un fabricant remonte "ModelX" et "Modelx" en considerant qu'il s'agit de 2 modeles differents. La string est codée dans le standard Zigbee ?

tcharp38 commented 4 years ago

Bon ecoute j'ai creusé un peu et je crois que cette histoire n'est pas assez claire mais le point doit rester ouvert. En gros est ce qu'on peut considerer que le model identifier est "case insensitive". Si ca n'est pas le cas il faut revoir la structure des devices et ca n'est pas immediat. Je vais me debrouiller pour avoir un env case sensitive pour avancer.

KiwiHC16 commented 4 years ago

Je vais integre le second pour voir ce que je recupere. La solution c est que je vais faire une verrue pour ce cas particulier dans le code et enlever le second modele qui pose probleme.

KiwiHC16 commented 4 years ago

Ok fait dans master.

KiwiHC16 commented 4 years ago

Ca doit resoudre ton probleme maintenant.

tcharp38 commented 4 years ago

Oui merci. J'ai du recloner le GIT mais la c'est propre. Je clos le point. Neanmoins peut etre faut il garder une issue ouverte ? Qu'en penses tu ?

KiwiHC16 commented 4 years ago

Je cherche à limiter le nombre d'issue au strict minimum avec des points précis car la liste est longue et on ne sait plus ce qui est important à faire. Je clos et et si on a un point precis on ouvrira une issue specifique.

tcharp38 commented 4 years ago

Oui parfait pour moi. C'était mon seul moyen de com avec toi.