-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
brand logo #83
Comments
Great idea. Feel free to make a pull request with the required changes. :)
28 Mar 2023 11:13:03 Arrheniu2 ***@***.***>:
…
Hi, im not sure if is an issue but we cant see the brand sems logo in the integration, maybe we can add it to the home assistants brand logo repositore?
not sure if we can pull that logo there to the domain "sems" or maybe we should host locally or add to this repositore
—
Reply to this email directly, view it on GitHub[#83], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAEQZ3F5GWGDOCSNNZCJ4FTW6KTQ5ANCNFSM6AAAAAAWKIWDSQ].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AAEQZ3CLC7RKC6MTW5NHMLLW6KTQ5A5CNFSM6AAAAAAWKIWDSSWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHGD5RTMI.gif]
|
i made my first github pull request here: probably i messed up something but eventually we will get the logo in home assistant (i hope) thanks for your work, tho. |
I guess you should resubmit that PR with the new Goodwe brand logo which is now a red stylized letter 'W'. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, im not sure if is an issue but we cant see the brand sems logo in the integration, maybe we can add it to the home assistants brand logo repositore?
not sure if we can pull that logo there to the domain "sems" or maybe we should host locally or add to this repositore
The text was updated successfully, but these errors were encountered: