Skip to content

SPH_MIX, should work for MIN_TLX but can't test#7

Closed
GraemeDBlue wants to merge 141 commits into
johanzander:masterfrom
GraemeDBlue:master
Closed

SPH_MIX, should work for MIN_TLX but can't test#7
GraemeDBlue wants to merge 141 commits into
johanzander:masterfrom
GraemeDBlue:master

Conversation

@GraemeDBlue
Copy link
Copy Markdown

No description provided.

@GraemeDBlue GraemeDBlue marked this pull request as draft November 7, 2025 16:22
@GraemeDBlue
Copy link
Copy Markdown
Author

had to revert this as cannot deploy as is to my main HA server as cannot force update growattServer it seems despite adding it in manifest

@GraemeDBlue GraemeDBlue force-pushed the master branch 3 times, most recently from 4fa05d8 to 79925bd Compare November 7, 2025 16:38
@GraemeDBlue
Copy link
Copy Markdown
Author

had to add the growattServer in as a dir and cant get the version to install

@GraemeDBlue
Copy link
Copy Markdown
Author

@johanzander this now works for SPH with ability to set charge and discharge windows and set power etc I cannot test for your tlx setup but I think i have isolated them enough

@johanzander
Copy link
Copy Markdown
Owner

johanzander commented Nov 10, 2025 via email

@GraemeDBlue
Copy link
Copy Markdown
Author

I have a branch in my upstream version where i can test your additions. You could as well if you want but it’s not tested at all yet mån 10 nov. 2025 kl. 12:04 skrev GraemeD @.>:

GraemeDBlue left a comment (johanzander/growatt_server_upstream#7) <#7 (comment)> @johanzander https://github.com/johanzander this now works for SPH with ability to set charge and discharge windows and set power etc I cannot test for your tlx setup but I think i have isolated them enough — Reply to this email directly, view it on GitHub <#7 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQSFAEXCGI7JYKBNO3TFAT34BWLRAVCNFSM6AAAAACLONMH6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTKMJQHA4TKNZZGU . You are receiving this because you were mentioned.Message ID: @.
>

Hi, yeah i looked at your sph_mix_support branch and used some of it as basis, but I had pushed some newer changes to growatServer itself so i needed tt make additional changes not in your branch

@GraemeDBlue
Copy link
Copy Markdown
Author

@johanzander it turned out this https://github.com/johanzander/growatt_server_upstream/blob/sph_mix_support/custom_components/growatt_server/manifest.json#L10 syntax was not installing the correct version on my main HAOS version of my HA install and that branch of mine you referenced is no longer valid

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