Skip to content

Add role 300 to SUPPORTED_ROLES (end-user account, v1 API) - #116

Merged
sHedC merged 1 commit into
sHedC:mainfrom
yaakov25:main
Oct 25, 2026
Merged

sHedC merged 1 commit into
sHedC:mainfrom
yaakov25:main

Conversation

@yaakov25

Copy link
Copy Markdown
Contributor

The Home Assistant integration fails with MasterthermUnsupportedRole: Role is not supported by this integration for my account.
My setup: pre-2022 heat pump (v1 API, mastertherm.vip-it.cz). Login via the official web portal and mobile app works fine. The login response (sanitized) is:

{"returncode":0,"message":"User logged in","modules":[{"id":"","module_name":"","config":[{"mb_addr":"1","mb_name":"unknown"}]}],"role":"300"}

This is a regular end-user account with a single own heat pump, so I believe 300 is a valid end-user role and should be accepted in SUPPORTED_ROLES. Happy to provide more details or testing if needed.

Signed-off-by: yaakov25 <yaakov@web.de>
@sHedC sHedC self-assigned this Oct 25, 2026
@sHedC
sHedC self-requested a review October 25, 2026 19:01
@codecov

codecov Bot commented Oct 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.65%. Comparing base (8bcf838) to head (a61c4a2).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   89.65%   89.65%           
=======================================
  Files          10       10           
  Lines         764      764           
=======================================
  Hits          685      685           
  Misses         79       79           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sHedC
sHedC merged commit ed5db26 into sHedC:main Oct 25, 2026
18 checks passed
@yaakov25

Copy link
Copy Markdown
Contributor Author

@sHedC I tested with the patch and new role and it now works. So ready to release. Thank you!

@sHedC

sHedC commented Oct 27, 2026

Copy link
Copy Markdown
Owner

great thanks having some validation issues should resolve soon, every time they update HASS something does not work quite right :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants