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

[basicui] Fix setpoint/slider widgets when %unit% is used in state pattern #1612

Merged
merged 1 commit into from
Dec 31, 2022

Conversation

lolodomo
Copy link
Contributor

@lolodomo lolodomo commented Dec 26, 2022

This is the same fix as #1611 but for OH 4.0 main branch.

Signed-off-by: Laurent Garnier lg.hc@free.fr

@lolodomo lolodomo added bug Something isn't working basic ui Basic UI labels Dec 26, 2022
@lolodomo lolodomo requested a review from a team as a code owner December 26, 2022 23:11
@lolodomo lolodomo changed the title basicui] Fix setpoint/slider widgets when %unit% is used in state [basicui] Fix setpoint/slider widgets when %unit% is used in state Dec 26, 2022
pattern

This is the same fix as openhab#1611 but for OH 4.0 main branch.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo
Copy link
Contributor Author

This PR will have to be cherry picked in branch 3.4.x as it fixed a critical issue introduced by a change in the core framework.

@lolodomo lolodomo added the patch A PR that has been cherry-picked to a patch release branch label Dec 31, 2022
@lolodomo
Copy link
Contributor Author

@kaikreuzer @ghys : please review this PR, we need it for the 3.4 patch (I added the tag).

@kaikreuzer kaikreuzer changed the title [basicui] Fix setpoint/slider widgets when %unit% is used in state [basicui] Fix setpoint/slider widgets when %unit% is used in state pattern Dec 31, 2022
@kaikreuzer kaikreuzer merged commit d6390d0 into openhab:main Dec 31, 2022
kaikreuzer pushed a commit that referenced this pull request Dec 31, 2022
)

This is the same fix as #1611 but for OH 4.0 main branch.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@kaikreuzer kaikreuzer added this to the 4.0 milestone Dec 31, 2022
@lolodomo lolodomo deleted the getUnitForWidget_OH4 branch December 31, 2022 16:59
@kaikreuzer
Copy link
Member

Thank you. I have cherry-picked the change onto the 3.4.x branch.

crnjan pushed a commit to crnjan/openhab-webui that referenced this pull request Jan 17, 2023
…enhab#1612)

This is the same fix as openhab#1611 but for OH 4.0 main branch.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Boris Krivonog <boris.krivonog@inova.si>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
basic ui Basic UI bug Something isn't working patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants