cmcgerty / Sofar2mqtt

A smart home interface for Sofar solar and battery inverters.
GNU General Public License v3.0
79 stars 45 forks source link

readme should say subscribe Sofar2mqtt/state - capital S #17

Closed rustle118 closed 2 years ago

rustle118 commented 2 years ago

Sofar2mqtt/state currently readme has lower case s for sofar

cmcgerty commented 2 years ago

Quite so. My bad. The mqtt message root is actually whatever you call the device in the line deviceName = "Sofar2mqtt";

Thanks, Colin


From: rustle118 @.> Sent: Friday, February 11, 2022 8:53:51 PM To: cmcgerty/Sofar2mqtt @.> Cc: Subscribed @.***> Subject: Re: [cmcgerty/Sofar2mqtt] readme should say subscribe Sofar2mqtt - capital S (Issue #17)

Sofar2mqtt/state currently readme has lower case s for sofar

— Reply to this email directly, view it on GitHubhttps://github.com/cmcgerty/Sofar2mqtt/issues/17#issuecomment-1036613774, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADFHULJBUATGSVJCPQHUKQTU2VZN7ANCNFSM5OFKB5EA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you are subscribed to this thread.Message ID: @.***>

cmcgerty commented 2 years ago

V2.0 beta1 has been published today, which is a major write and bring much greater stability. See if that is better for you.