Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[P176] Add plugin Communication - Victron VE.Direct #5148

Open
wants to merge 24 commits into
base: mega
Choose a base branch
from
Open

[P176] Add plugin Communication - Victron VE.Direct #5148

wants to merge 24 commits into from

Conversation

tonhuisman
Copy link
Contributor

@tonhuisman tonhuisman commented Oct 25, 2024 •
edited
Loading

From a forum request

Feature:

  • Plugin P176 - Communication - Victron VE.Direct
  • Add variables successcount, errorcount and updated
  • Add setting for Events only on updated data, to only send events and Controller data when new data is received

TODO:

  • Testing by requester
    • Validate checksum handling (confirmed)
    • Test Events only on updated data feature
  • Add documentation

@tonhuisman tonhuisman mentioned this pull request Oct 25, 2024
Open
tonhuisman and others added 18 commits October 26, 2024 14:43
@tonhuisman
Merge branch 'mega' of https://github.com/letscontrolit/ESPEasy into …
9dd301b 
…feature/P176-add-plugin-victron-ve.direct
@tonhuisman
[P176] Add some features and code improvements
0aecf3b
@tonhuisman
[P176] Fix ESP8266 build issue
9d8135e
@tonhuisman
[P176] Update taskvalues at receiving data, other improvements
d67d8e6
@tonhuisman
[P176] Fix use of plugin compiletime feature flags
78d6a2c
@tonhuisman
Merge branch 'mega' of https://github.com/letscontrolit/ESPEasy into …
8c3eb08 
…feature/P176-add-plugin-victron-ve.direct
@tonhuisman
[P176] Move Led initialization to DataStruct initialization function
f5068aa
@tonhuisman
[P176] Add documentation
3e862bc
@tonhuisman
[P176] Refactorings to optimize performance (lower load), remove RX T…
7a5f744 
…imeout setting
@tonhuisman
[P176] Update documentation
9ca8f79
@TD-er
[P176] Simplify handling temporary values before checksum checked
88f7940
@tonhuisman
[P176] Avoid calling serial->available() too often, uncrustify some r…
f2a01e6 
…ecent changes
@tonhuisman
[P176] Update changelog (and GH Actions failed)
8f475f1
@tonhuisman
[P176] Handle ON and OFF values as 1 and 0
21ff328
@tonhuisman
[P176] Fix checksum validation, was ignoring the first line after rec…
7de795e 
…eiving a valid checksum
@TD-er
Merge branch 'mega' into feature/P176-add-plugin-victron-ve.direct
c8d6e45
@tonhuisman
Merge branch 'mega' of https://github.com/letscontrolit/ESPEasy into …
ed33fd0 
…feature/P176-add-plugin-victron-ve.direct
@tonhuisman
[P176] Add variables successcount, errorcount and ischanged
7954406
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TD-er TD-er TD-er left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tonhuisman @TD-er