Skip to content

Script for victron mppt solar controller#168

Merged
taulfsime merged 1 commit intomainfrom
smart-solar-mppt-charge-controller
Nov 17, 2025
Merged

Script for victron mppt solar controller#168
taulfsime merged 1 commit intomainfrom
smart-solar-mppt-charge-controller

Conversation

@Mila5847
Copy link
Copy Markdown
Collaborator

No description provided.

@Mila5847 Mila5847 force-pushed the smart-solar-mppt-charge-controller branch from 0946bfe to f863d10 Compare November 17, 2025 10:21
@Mila5847 Mila5847 force-pushed the smart-solar-mppt-charge-controller branch from f863d10 to 107f0da Compare November 17, 2025 10:24
@Mila5847 Mila5847 requested a review from taulfsime November 17, 2025 10:25
@Mila5847 Mila5847 assigned Mila5847 and unassigned taulfsime Nov 17, 2025
Copy link
Copy Markdown
Collaborator

@taulfsime taulfsime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good job

@taulfsime taulfsime merged commit 9c5b23a into main Nov 17, 2025
@MetaChuh
Copy link
Copy Markdown

@Mila5847
works like a charm, thank you so much for this.

one note: function scanCB(ev, res) is declared twice, but it still works and only the second declared scanCB function is interpreted.

big kudos, many thx & greetings,
metachuh

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.

3 participants