Dev device tracker 1 9 0 - replace PR 84

cyr-ius / hass-livebox-component

Livebox Component for Home assistant

MIT License

52 stars 21 forks source link

Dev device tracker 1 9 0 - replace PR 84 #91

Closed LGO44 closed 5 months ago

LGO44 commented 5 months ago

update the file device tracker from the version 1.9.0

cyr-ius commented 5 months ago

Please check review recommandatios

LGO44 commented 5 months ago

Please check review recommandatios

I'm not a confirmed developper, I don't see what i have to do to solved this problem. Could you help me?

cyr-ius commented 5 months ago

Please check review recommandatios

I'm not a confirmed developper, I don't see what i have to do to solved this problem. Could you help me?

Je vois que tu t'es perdu dans le Pull Request sur github. Si tu veux faire des modifications , c'est très simple.. tu reste sur la branche que tu dévellope de ton côté, tu corriges une ou plusieurs remarque , tu fais des commit et ensuite tu fais ton push pour pousser sur ta branch vers Github. comme ton PullRequest est issue de cette braneche , cela va automatiquement mettre à jour le PR que tu m'as proposer Il suffit ensuite d'aller sur sur mes remarques et de les passer en Resolve

cyr-ius commented 5 months ago

Si de mon coté , je fais des modifs de ton code , en fait je modifie la branch de ton dépot Github. Il sufit de faire un git pull pour tirer mes modifs en loca sur ton pc.

Si j'ai modifié un page sur lequel tu trvailles en même temps.. il faut faire un git pull --rebase. Ainsi Git va te proposer de choisir quel version tu gardes. C'est pour cela que bien souvent on fait toujours un git pull --rebase au lieu d'un git pull

LGO44 commented 5 months ago

Je vois que tu t'es perdu dans le Pull Request sur github.

c'est un euphémisme, je ne vois pas tes remarques

cyr-ius commented 5 months ago

Par exemple , là je viens d'effacer le commentaire : " #added by LGO " et bien si tu regardes sur ton github dans ta branche "dev_device_tracker_1_9_0" , la ligne à disparu et tu as commit de ma part.

cyr-ius commented 5 months ago

Je vois que tu t'es perdu dans le Pull Request sur github.

c'est un euphémisme, je ne vois pas tes remarques

Mes remarque appraisent juste dans ce billet. Regarde un peu plus haut . Tu vois des bour de code ,avec mes remarques. Cela permet de t e pointer un morceau de code et d'échanger sur cette partie .

Dans ton dépot , tu vois pas les remarques , elles sont stocker uniquement dans cette fiche de PR

cyr-ius commented 5 months ago

Comme je viens de mofier ta page device_tracker en supprimant ton commentaire. Tu vas donc avoir ta page plus récente sur ton github que sur ton pc. Il faut donc faire un git pull --rebase ou simplement git pull pour récupérer la mise en jour en local

LGO44 commented 5 months ago

Il me reste un conflit dans le fichier manifest.json je ne comprends pas....

LGO44 commented 5 months ago

Quelle est la suite de la procédure?

cyr-ius commented 5 months ago

je suis en pleine refonte de l Addon. Du coup si les tests de la version 2.0.0-beta sont concluant, j embarque tes évolutions dans la version 2.1.0-beta

Si tu veux participer aux tests de la 2.0.0-beta , il suffit d activer le mode beta dans HACS et noter le retour dans #93

LGO44 commented 5 months ago

Bonsoir, Pour le test de la nouvelle version c'est biensur OK j'ai essayé la beta 2.0.3

Sur un nouveau Home assistant RPI5 la connexion echoue systématiquement
J'ai supprimé cette version puis installé et configure la version 1.9.0
J'ai réinstallé la version beta 2.0.3, la configuration échoué

ci joint un fichier de log avec

le message d'erreur périodique
les messages suite à une demande de configuration

Erreur lors du scan de la livebox.txt

cyr-ius commented 5 months ago

Voir la version 2.1.2-beta