From 8b64671151e110b3479030decdcfeb7924f60199 Mon Sep 17 00:00:00 2001
From: helgeerbe <helge@erbehome.de>
Date: Tue, 23 May 2023 11:04:56 +0200
Subject: [PATCH] doc: warning,  that only 5 inverters are supported at max

---
 README.md                 | 4 ++++
 README_onBattery.md       | 4 ++++
 src/WebApi_prometheus.cpp | 2 +-
 src/WebApi_ws_live.cpp    | 2 +-
 4 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 8fe44c4f..11f3ac0e 100644
--- a/README.md
+++ b/README.md
@@ -13,6 +13,10 @@ This is a fork from the Hoymiles project OpenDTU.
 
 This project is still under development and adds following features:
 
+> **Warning**
+> 
+> In contrast to the original openDTU, with release 2023.05.23.post1 openDTU-onBattery supports only 5 inverters. Otherwise, there is not enough memory for the liveData view.
+
 * Support Victron's Ve.Direct protocol on the same chip (cable based serial interface!). Additional information about Ve.direct can be downloaded from https://www.victronenergy.com/support-and-downloads/technical-information.
 * Dynamically sets the Hoymiles power limited according to the currently used energy in the household. Needs an HTTP JSON based power meter (e.g. Tasmota), an MQTT based power meter like Shelly 3EM or an SDM power meter.
 * Battery support: Read the voltage from Victron MPPT charge controller or from the Hoymiles DC inputs and starts/stops the power producing based on configurable voltage thresholds
diff --git a/README_onBattery.md b/README_onBattery.md
index a1161537..a9b824b5 100644
--- a/README_onBattery.md
+++ b/README_onBattery.md
@@ -4,6 +4,10 @@ This is a fork from the Hoymiles project [OpenDTU](https://github.com/tbnobody/O
 
 ![GitHub tag (latest SemVer)](https://img.shields.io/endpoint?url=https://gist.githubusercontent.com/helgeerbe/68b47cc8c8994d04ab3a4fa9d8aee5e6/raw/openDTUcoreRelease.json)
 
+> **Warning**
+> 
+> In contrast to the original openDTU, with release 2023.05.23.post1 openDTU-onBattery supports only 5 inverters. Otherwise, there is not enough memory for the liveData view.
+
 ## Features
 
 * Hoymiles inverter support for live data and display of various inverter internal information. (Partial) support for multiple inverters.
diff --git a/src/WebApi_prometheus.cpp b/src/WebApi_prometheus.cpp
index 274e3e5e..afbea75f 100644
--- a/src/WebApi_prometheus.cpp
+++ b/src/WebApi_prometheus.cpp
@@ -27,7 +27,7 @@ void WebApiPrometheusClass::loop()
 void WebApiPrometheusClass::onPrometheusMetricsGet(AsyncWebServerRequest* request)
 {
     try {
-        auto stream = request->beginResponseStream("text/plain; charset=utf-8", 4096 * INV_MAX_COUNT); // TODO (helge) check if this calculation is correct
+        auto stream = request->beginResponseStream("text/plain; charset=utf-8", 4096 * INV_MAX_COUNT); // TODO(helge) check if this calculation is correct
 
         stream->print("# HELP opendtu_build Build info\n");
         stream->print("# TYPE opendtu_build gauge\n");
diff --git a/src/WebApi_ws_live.cpp b/src/WebApi_ws_live.cpp
index 1fb3b055..7a359358 100644
--- a/src/WebApi_ws_live.cpp
+++ b/src/WebApi_ws_live.cpp
@@ -69,7 +69,7 @@ void WebApiWsLiveClass::loop()
             String buffer;
             // free JsonDocument as soon as possible
             {
-                DynamicJsonDocument root(4096 * INV_MAX_COUNT); // TODO (helge) check if this calculation is correct
+                DynamicJsonDocument root(4096 * INV_MAX_COUNT); // TODO(helge) check if this calculation is correct
                 JsonVariant var = root;
                 generateJsonResponse(var);
                 serializeJson(root, buffer);