-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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.
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
Update Lixee.ts, need help #8728
base: master
Are you sure you want to change the base?
Conversation
* add tariff period new value * fix chunk default description * change default chunk value to 2
src/devices/lixee.ts
Outdated
const local = { | ||
modernExtend: { | ||
addCustomClusterManuSpecificLixee: () => | ||
deviceAddCustomCluster('liXeePrivate', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deviceAddCustomCluster('liXeePrivate', { | |
deviceAddCustomCluster('liXeePrivateCustom', { |
Could you check if this now work from the dev console? Otherwise there is a conflict with the already existing cluster.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok thank you for this
Where do you get this? |
So I don't know how to test.
Or others but I don't know if it's linked to js file, if i have to use tz file ... |
Try changing |
Thank you for the help. I removed all the typescript stuff and it's works now but how to compile to ts file ? other question : I tried to enhance some exposes datas with composite and features but that's add an "apply button" how to do the same without button ? I didn't find the equivalent. in attachment the lixee.js file (in txt format) |
Composite always has an apply button (since all attributes need to be submitted at once). |
I tried to update the converters but I don't know how to test in local.
Can anyone help ?