Skip to content

Allow empty discoveryId#110

Open
m6w6 wants to merge 1 commit intozivillian:masterfrom
m6w6:m6w6-patch-1
Open

Allow empty discoveryId#110
m6w6 wants to merge 1 commit intozivillian:masterfrom
m6w6:m6w6-patch-1

Conversation

@m6w6
Copy link
Copy Markdown

@m6w6 m6w6 commented May 8, 2024

I was wondering for quite some time, why nothing really worked anymore.
Looks like the fix for #101 (i.e. ignoring empty discoveryID) killed it for me.
Would it be possible to implement something like what I dabbled together in this PR?

@zivillian
Copy link
Copy Markdown
Owner

Moin, sorry für meine späte Reaktion - ich war ein paar Wochen offline.

Grundsätzlich spricht nichts dagegen, aber ich würde gern sichergehen, dass ich das Problem verstehe (deshalb jetzt auch auf deutsch) und die Auswirkung auf alle anderen User so gering wie möglich halten.

In #103 sieht es so aus, als würden die zusätzlichen Parameter dein Problem lösen - oder hab ich den zu früh geschlossen?

Wie genau sieht dein MQTT Tree aktuell aus, und wie soll er aussehen?

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

Successfully merging this pull request may close these issues.

2 participants