Prepare Release 2024.10.22

hoylabs · Oct 22, 2024 · 225a66a · 225a66a
2 parents 83ed72e + 3aa850a
commit 225a66a
Show file tree

Hide file tree

Showing 161 changed files with 6,825 additions and 3,921 deletions.
diff --git a/.github/ISSUE_TEMPLATE/bug_report.yml b/.github/ISSUE_TEMPLATE/bug_report.yml
@@ -4,19 +4,19 @@ labels: ["bug"]
 body:
   - type: markdown
     attributes:
-      value: >
-        ### ⚠️ Please remember: issues are for *bugs*
-        That is, something you believe affects every single user of OpenDTU, not just you. If you're not sure, start with one of the other options below.
+      value: |
+        ### ⚠️ Please remember: issues are for *bugs*⚠️
+        That is, something you believe affects every single user of OpenDTU-OnBattery, not just you. If you're not sure, start with one of the other options below.
   - type: markdown
     attributes:
       value: |
-        #### Have a question? 👉 [Start a new discussion](https://github.com/tbnobody/OpenDTU/discussions/new) or [ask in chat](https://discord.gg/WzhxEY62mB).
+        #### Have a question? 👉 [Start a new discussion](https://github.com/helgeerbe/OpenDTU-OnBattery/discussions/new/choose) or [ask in chat](https://discord.gg/WzhxEY62mB).
 
         #### Before opening an issue, please double check:
 
-        - [Documentation](https://www.opendtu.solar).
-        - [The FAQs](https://www.opendtu.solar/firmware/faq/).
-        - [Existing issues and discussions](https://github.com/tbnobody/OpenDTU/search?q=&type=issues).
+        - [Documentation](https://opendtu-onbattery.net)
+        - [The FAQs](https://opendtu-onbattery.net/firmware/faq/)
+        - [Existing issues and discussions](https://github.com/helgeerbe/OpenDTU-OnBattery/search?q=&type=issues)
   - type: textarea
     id: what-happened
     attributes:
@@ -45,20 +45,37 @@ body:
     id: install_format
     attributes:
       label: Install Method
-      description: How did you install OpenDTU?
+      description: How did you install OpenDTU-OnBattery?
       options:
-        - Pre-Compiled binary from GitHub
+        - Pre-Compiled binary from GitHub releases
+        - Pre-Compiled binary from GitHub actions/pull-request
         - Self-Compiled
     validations:
       required: true
   - type: input
     id: version
     attributes:
-      label: What git-hash/version of OpenDTU?
-      description: You can find this in by going to Info -> System
+      label: What git-hash/version of OpenDTU-OnBattery?
+      description: You can find this in the Web UI at Info -> System.
       placeholder: "e.g. 359d513"
     validations:
       required: true
+  - type: dropdown
+    id: environment
+    attributes:
+      label: What firmware variant (PIO Environment)?
+      description: You can find this in the Web UI at Info -> System.
+      options:
+          - "generic_esp32s3_usb"
+          - "generic_esp32s3"
+          - "generic_esp32_8mb"
+          - "generic_esp32_4mb_no_ota"
+          - "generic_esp32"
+          - "generic"
+          - "opendtufusionv2"
+          - "other (tell us in 'Anything else?')"
+    validations:
+      required: true
   - type: textarea
     id: logs
     attributes:
@@ -78,11 +95,11 @@ body:
     attributes:
       label: Please confirm the following
       options:
-        - label: I believe this issue is a bug that affects all users of OpenDTU, not something specific to my installation.
+        - label: I believe this issue is a bug that affects all users of OpenDTU-OnBattery, not something specific to my installation.
           required: true
         - label: I have already searched for relevant existing issues and discussions before opening this report.
           required: true
         - label: I have updated the title field above with a concise description.
           required: true
-        - label: I have double checked that my inverter does not contain a W in the model name (like HMS-xxxW) as they are not supported
+        - label: I have double checked that my inverter does not contain a W in the model name (like HMS-xxxW) as they are not supported.
           required: true
diff --git a/.github/ISSUE_TEMPLATE/config.yml b/.github/ISSUE_TEMPLATE/config.yml
@@ -4,5 +4,5 @@ contact_links:
     url: https://discord.gg/WzhxEY62mB
     about: Discuss with us on Discord
   - name: 🤔 Have questions or need support?
-    url: https://github.com/tbnobody/OpenDTU/discussions
+    url: https://github.com/helgeerbe/OpenDTU-OnBattery/discussions
     about: Use the GitHub Discussions feature
diff --git a/.github/ISSUE_TEMPLATE/feature_request.yml b/.github/ISSUE_TEMPLATE/feature_request.yml
@@ -1,12 +1,12 @@
 name: ✨ Request a feature
-description: Suggest an improvement idea for OpenDTU!
+description: Suggest an improvement idea for OpenDTU-OnBattery!
 title: "[Request]"
 labels: ["enhancement"]
 body:
   - type: markdown
     attributes:
       value: >
-        **Thank you for wanting to request a feature in OpenDTU!**
+        **Thank you for wanting to request a feature in OpenDTU-OnBattery!**
 
         Before you go ahead with your request, please first consider if it wouldn't be
         better suited in a external home automation software like OpenHAB, ioBroker, Home Assistant etc.

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -48,7 +48,7 @@ jobs:
       environments: ${{ steps.envs.outputs.environments }}
 
   build:
-    name: Build Enviornments
+    name: Build Environments
     runs-on: ubuntu-latest
     needs: get_default_envs
     strategy:
@@ -93,18 +93,27 @@ jobs:
           python -m pip install --upgrade pip
           pip install --upgrade platformio setuptools
 
+      - name: Enable Corepack
+        run: |
+            cd webapp
+            corepack enable
+
       - name: Setup Node.js and yarn
         uses: actions/setup-node@v4
         with:
-          node-version: "20"
+          node-version: "22"
           cache: "yarn"
           cache-dependency-path: "webapp/yarn.lock"
 
       - name: Install WebApp dependencies
-        run: yarn --cwd webapp install --frozen-lockfile
+        run: |
+            cd webapp
+            yarn install --frozen-lockfile
 
       - name: Build WebApp
-        run: yarn --cwd webapp build
+        run: |
+            cd webapp
+            yarn build
 
       - name: Build firmware
         run: pio run -e ${{ matrix.environment }}
@@ -130,7 +139,7 @@ jobs:
     if: startsWith(github.ref, 'refs/tags/2')
     steps:
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Get tags
         run: git fetch --force --tags origin

diff --git a/.github/workflows/config/release-notes-config.json b/.github/workflows/config/release-notes-config.json
@@ -18,6 +18,12 @@
                 "fix"
             ]
         },
+        {
+            "title": "## 🌎 Web Application",
+            "labels": [
+                "webapp"
+            ]
+        },
         {
             "title": "## 📚 Documentation",
             "labels": [

diff --git a/.github/workflows/cpplint.yml b/.github/workflows/cpplint.yml
@@ -6,16 +6,22 @@ jobs:
   build:
     runs-on: ubuntu-latest
 
+    # prevent push event from triggering if it's part of a local PR, see
+    # https://github.com/orgs/community/discussions/57827#discussioncomment-6579237
+    if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name
+
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Set up Python
         uses: actions/setup-python@v5
         with:
           python-version: "3.x"
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          pip install cpplint
+          pip install cpplint==1.6.1
       - name: Linting
         run: |
-          cpplint --repository=. --recursive --filter=-build/c++11,-runtime/references,-readability/braces,-whitespace,-legal,-build/include ./src ./include ./lib/Hoymiles ./lib/MqttSubscribeParser ./lib/TimeoutHelper ./lib/ResetReason
+          cpplint --repository=. --recursive \
+            --filter=-build/c++11,-runtime/references,-readability/braces,-whitespace,-legal,-build/include \
+            ./src ./include ./lib/Hoymiles ./lib/MqttSubscribeParser ./lib/TimeoutHelper ./lib/ResetReason
diff --git a/.github/workflows/test_build.yml b/.github/workflows/test_build.yml
diff --git a/.github/workflows/yarnlint.yml b/.github/workflows/yarnlint.yml
@@ -6,17 +6,27 @@ jobs:
   build:
     runs-on: ubuntu-latest
 
+    # prevent push event from triggering if it's part of a local PR, see
+    # https://github.com/orgs/community/discussions/57827#discussioncomment-6579237
+    if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name
+
+    defaults:
+      run:
+        working-directory: webapp
+
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
+      - name: Enable Corepack
+        run: corepack enable
       - name: Setup Node.js and yarn
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
-          node-version: "18"
+          node-version: "22"
           cache: "yarn"
           cache-dependency-path: "webapp/yarn.lock"
 
       - name: Install WebApp dependencies
-        run: yarn --cwd webapp install --frozen-lockfile
+        run: yarn install --frozen-lockfile
 
       - name: Linting
-        run: yarn --cwd webapp lint
+        run: yarn lint
diff --git a/.github/workflows/yarnprettier.yml b/.github/workflows/yarnprettier.yml
@@ -0,0 +1,32 @@
+name: Yarn Prettier
+
+on: [push, pull_request]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    # prevent push event from triggering if it's part of a local PR, see
+    # https://github.com/orgs/community/discussions/57827#discussioncomment-6579237
+    if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name
+
+    defaults:
+      run:
+        working-directory: webapp
+
+    steps:
+      - uses: actions/checkout@v4
+      - name: Enable Corepack
+        run: corepack enable
+      - name: Setup Node.js and yarn
+        uses: actions/setup-node@v4
+        with:
+          node-version: "22"
+          cache: "yarn"
+          cache-dependency-path: "webapp/yarn.lock"
+
+      - name: Install WebApp dependencies
+        run: yarn install --frozen-lockfile
+
+      - name: Check Formatting
+        run: yarn prettier --check src/