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

[homeconnect] Predefined temp / spin speeds options for unsupported washer programs #10953

Merged
merged 3 commits into from
Jul 25, 2021

Conversation

lolodomo
Copy link
Contributor

@lolodomo lolodomo commented Jul 6, 2021

Fix #10701

Also save in programs cache the unuspported program

Signed-off-by: Laurent Garnier [email protected]

@lolodomo lolodomo added enhancement An enhancement or new feature for an existing add-on work in progress A PR that is not yet ready to be merged awaiting other PR Depends on another PR labels Jul 6, 2021
@lolodomo lolodomo requested a review from bruestel as a code owner July 6, 2021 18:18
@lolodomo
Copy link
Contributor Author

lolodomo commented Jul 6, 2021

This PR is in fact ready but as code is based on PR #10929, we need to first wait for the merge of the other PR.

@lolodomo lolodomo force-pushed the unsupportedprogramoptions branch from ac46039 to 35b6e35 Compare July 11, 2021 21:11
@lolodomo lolodomo removed the awaiting other PR Depends on another PR label Jul 11, 2021
@lolodomo lolodomo force-pushed the unsupportedprogramoptions branch from 35b6e35 to a34e00f Compare July 12, 2021 14:05
@lolodomo lolodomo removed the work in progress A PR that is not yet ready to be merged label Jul 12, 2021
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/home-connect-addon-looking-for-beta-testers/49702/890

@lolodomo
Copy link
Contributor Author

@bruestel : this enhancement is now finished and ready for a review. So if you have time and availability...

@lolodomo lolodomo force-pushed the unsupportedprogramoptions branch from a34e00f to 24ed800 Compare July 13, 2021 11:22
…asher programs

Fix openhab#10701

Also save in programs cache the unuspported program

Signed-off-by: Laurent Garnier <[email protected]>
@lolodomo lolodomo force-pushed the unsupportedprogramoptions branch from 24ed800 to 09bf3e2 Compare July 13, 2021 11:28
@lolodomo lolodomo added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Jul 19, 2021
Copy link
Contributor

@bruestel bruestel left a comment

Choose a reason for hiding this comment

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

Thanks for your afford. Code looks good, unfortunately I cannot test it on a real device.

Copy link
Contributor

@bruestel bruestel left a comment

Choose a reason for hiding this comment

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

To make sure that your code improvements don't break anything I've also tested your code changes against the simulator (dishwasher and washer). Works as expected.

@lolodomo
Copy link
Contributor Author

@fwolter : could you please review ?

@lolodomo lolodomo requested a review from fwolter July 25, 2021 07:56
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.

LGTM

@fwolter fwolter merged commit 3d0c31b into openhab:main Jul 25, 2021
@fwolter fwolter added this to the 3.2 milestone Jul 25, 2021
@lolodomo lolodomo deleted the unsupportedprogramoptions branch July 25, 2021 10:00
@lolodomo
Copy link
Contributor Author

Thank you @fwolter

frederictobiasc pushed a commit to frederictobiasc/openhab-addons that referenced this pull request Oct 26, 2021
…asher programs (openhab#10953)

* [homeconnect] Predefined temp / spin speeds options for unsupported washer programs

Fix openhab#10701

Also save in programs cache the unuspported program

Signed-off-by: Laurent Garnier <[email protected]>

* Use constants OPTION_WASHER_TEMPERATURE and OPTION_WASHER_SPIN_SPEED

Signed-off-by: Laurent Garnier <[email protected]>

* Review comment : using constants

Signed-off-by: Laurent Garnier <[email protected]>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
…asher programs (openhab#10953)

* [homeconnect] Predefined temp / spin speeds options for unsupported washer programs

Fix openhab#10701

Also save in programs cache the unuspported program

Signed-off-by: Laurent Garnier <[email protected]>

* Use constants OPTION_WASHER_TEMPERATURE and OPTION_WASHER_SPIN_SPEED

Signed-off-by: Laurent Garnier <[email protected]>

* Review comment : using constants

Signed-off-by: Laurent Garnier <[email protected]>
dschoepel pushed a commit to dschoepel/openhab-addons that referenced this pull request Nov 9, 2021
…asher programs (openhab#10953)

* [homeconnect] Predefined temp / spin speeds options for unsupported washer programs

Fix openhab#10701

Also save in programs cache the unuspported program

Signed-off-by: Laurent Garnier <[email protected]>

* Use constants OPTION_WASHER_TEMPERATURE and OPTION_WASHER_SPIN_SPEED

Signed-off-by: Laurent Garnier <[email protected]>

* Review comment : using constants

Signed-off-by: Laurent Garnier <[email protected]>
Signed-off-by: Dave J Schoepel <[email protected]>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…asher programs (openhab#10953)

* [homeconnect] Predefined temp / spin speeds options for unsupported washer programs

Fix openhab#10701

Also save in programs cache the unuspported program

Signed-off-by: Laurent Garnier <[email protected]>

* Use constants OPTION_WASHER_TEMPERATURE and OPTION_WASHER_SPIN_SPEED

Signed-off-by: Laurent Garnier <[email protected]>

* Review comment : using constants

Signed-off-by: Laurent Garnier <[email protected]>
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.

[homeconnect] Spin speed / washing temperature for unsupported washing program
4 participants