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

incorrect units #113

Closed
Luapsi opened this issue Oct 31, 2023 · 1 comment
Closed

incorrect units #113

Luapsi opened this issue Oct 31, 2023 · 1 comment
Labels
bug Something isn't working fixed
Milestone
Release 1.1.x

Comments

@Luapsi
Copy link

Luapsi commented Oct 31, 2023

Units for two data fields are incorrect!

Expected behavior
The unit kWh is expected in the database field “value.power.production”.
The unit w is expected in the database field "value.power"
Screenshots & Logfiles

Versions:

  • Adapter version: 1.0.2
  • NPM version: 9.8.1
  • Node version: v18.18.2
  • Operating system: Windows10

2023-10-31_160724
@mcm1957 mcm1957 added the bug Something isn't working label Oct 31, 2023
mcm1957 added a commit that referenced this issue Nov 12, 2023
@mcm1957
fix incorrect units (#113)
5322053
@mcm1957 mcm1957 added the fixed label Nov 12, 2023
mcm1957 added a commit that referenced this issue Nov 12, 2023
@mcm1957
chore: release v1.1.0
c111bfa 
-   (mcm1957) Adapter requires nodejs 18 now.
-   (mcm1957) Incorrect energy units have been corrected. [#113]
-   (mcm1957) New internal information returned by api is no longer reported as unknown. [#110]
-   (mcm1957) Dependencies have been updated.
@mcm1957 mcm1957 added this to the Release 1.1.x milestone Nov 12, 2023
@mcm1957
Copy link
Member

mcm1957 commented Nov 12, 2023

fixed with 1.1.0

@mcm1957 mcm1957 closed this as completed Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working fixed
Projects
None yet
Milestone
Release 1.1.x
Development

No branches or pull requests
2 participants
@mcm1957 @Luapsi