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.

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

[modbus.e3dc] Add battery capacity #15085

Merged
merged 2 commits into from Jun 16, 2023
Merged

Conversation

weymann
Copy link
Contributor

@weymann weymann commented Jun 11, 2023

Resolves #13171

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
Copy link
Member

@fwolter fwolter left a comment

Choose a reason for hiding this comment

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

You could update the English18n properties file to make the new channels available for translation in Crowdin by running mvn org.openhab.core.tools:i18n-maven-plugin:3.4.0:generate-default-translations.

@weymann weymann requested a review from fwolter June 13, 2023 20:57
@fwolter
Copy link
Member

fwolter commented Jun 14, 2023

Can you comment on updating the i18n file?

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
@weymann
Copy link
Contributor Author

weymann commented Jun 15, 2023

Missed i18n - added!

@fwolter fwolter merged commit 03c5243 into openhab:main Jun 16, 2023
2 of 3 checks passed
@fwolter fwolter added the enhancement An enhancement or new feature for an existing add-on label Jun 16, 2023
@fwolter fwolter added this to the 4.0 milestone Jun 16, 2023
tb4jc pushed a commit to tb4jc/openhab-addons that referenced this pull request Jun 19, 2023
* battery capacity added

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

* i18n for channel additions

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

---------

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
Signed-off-by: Thomas Burri <thomas.burri@alstomgroup.com>
@jlaur
Copy link
Contributor

jlaur commented Jul 7, 2023

@weymann - update instructions are missing for the added channels - see https://next.openhab.org/docs/developer/bindings/thing-xml.html#updating-thing-types

@jlaur jlaur changed the title [modbus.e3dc] battery capacity added [modbus.e3dc] Add battery capacity Jul 7, 2023
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Aug 9, 2023
* battery capacity added

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

* i18n for channel additions

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

---------

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
Signed-off-by: Matt Myers <mmyers75@icloud.com>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
* battery capacity added

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

* i18n for channel additions

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

---------

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[modbus.e3dc] Battery Storage Support
3 participants