Skip to content

IQ battery als aparte device toegevoegd#3

Open
Wouwmw wants to merge 1 commit intoWeeJeWel:mainfrom
Wouwmw:patch02
Open

IQ battery als aparte device toegevoegd#3
Wouwmw wants to merge 1 commit intoWeeJeWel:mainfrom
Wouwmw:patch02

Conversation

@Wouwmw
Copy link
Copy Markdown
Contributor

@Wouwmw Wouwmw commented Jan 11, 2026

Nu heb ik bij installatie 2 devices en jouw code amper gewijzigd. Je ziet de nieuwe device met settings IQbattery / IQbat.
Nogmaals het is mijn eerste (geupdate) app via Visual Studio voor Homey.

Het voelt naar meer...

Nu heb ik bij installatie 2 devices.
import fetch from 'node-fetch';
import EnphaseDevice from '../../lib/EnphaseDevice.mjs';

export default class EnphaseDeviceInverter extends EnphaseDevice {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Incorrect class name

@@ -0,0 +1,5 @@
import EnphaseDriver from '../../lib/EnphaseDriver.mjs';

export default class EnphaseDriverInverter extends EnphaseDriver {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Incorrect class name

@@ -0,0 +1,14 @@
{
"id": "IQbat_charge",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please use lowercase for all custom capabilities.

@@ -0,0 +1,53 @@
{
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This file lives in homeycompose but I think it should be .homeycompose?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Incorrect image.

}
}
},
"capabilitiesConfig": {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This property doesn't exist.

},
"energy": {
"batteries": [
"OTHER"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

INTERNAL?

"name": "Emile Nijssen",
"email": "emile@athom.nl"
},
"contributors": {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please format all documents.

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