-
Notifications
You must be signed in to change notification settings - Fork 5
Fix invalid Device Info #44
Conversation
Device_reg_fix
|
The failed check seems to be related to translations. |
|
Hi, Könntest du mir eventuell ein paar Infos geben, welche Actions ich für den Workflow noch brauche oder einen Hinweis geben, wo ich was dazu finde? Danke und Grüße |
|
And BTW, is the release coming later automatically, or is it failing because of the failed check? |
Release is failing due to checks it seems. |
|
🎉 This PR is included in version 2.5.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Die Actions für die Checks kommen aus dem Template das ich ursprünglich für das Repo genutzt habe. Allerdings nutze ich für den automatischen Release semantic release (ich glaube der Workflow im Template hat damals noch keine automatischen Releases unterstützt). Wenn du das Repo auf GitHub forkst und im Fork GitHub Actions aktivierst, sollte eigentlich alles ohne weiteres Setup funktionieren. |
|
Danke, musste sie tatsächlich nur aktivieren. Ich habe mal nachgelesen und offensichtlich ist es nicht möglich die Issus in mein repo zu verschieben. Die Doku sagt "only between repos OWNED by the same account" - da reichen einfache schreibrechte offensichtlich nicht aus. Wenn ich die integration übernehme, dann sollten wir uns überlgen, das in die Readme deines repos aufzunehmen, damit die User es mitbekommen. Soweit die Gedanken zum Sonntag. |
|
Ich würde in der README auf dein Repo verlinken und mein Repo archivieren. |
|
OK, das klingt gut - ich habe mein Repo jetzt soweit eingestellt und dort die Version 2.6.0 erstellt. Das Verfahren zum updaten sähe wir folgt aus:
Damit sollte die Konfiguration erhalten bleiben (sagen zumindest meine Tests) Die issues habe ich alle kopiert und würde das nach Umstellung in den Kommentaren jeweils bekannt machen. Danke und Grüße |
|
Ich habe dein Repo in meiner README verlinkt. Ich schlage vor du packst den Guide zum Updaten in deine README. |


This simple change fixes the warning about invalid Device info, that will turn into errors with HA 2023.09.
Since this will happen tonight, it is quite urgent.
Background:
All the keys in deviceInfo must be from one of the categories. In this case Primary.
p.s.: Sorry for all the commit messages - don't know how to get rid of them.
Closes #42
Closes #45