Skip to content

Claude/sungrow api v1 migration fax xs#8

Closed
pirvu wants to merge 7 commits intoKRoperUK:mainfrom
pirvu:claude/sungrow-api-v1-migration-faxXs
Closed

Claude/sungrow api v1 migration fax xs#8
pirvu wants to merge 7 commits intoKRoperUK:mainfrom
pirvu:claude/sungrow-api-v1-migration-faxXs

Conversation

@pirvu
Copy link

@pirvu pirvu commented Mar 7, 2026

No description provided.

claude and others added 7 commits March 7, 2026 15:06
…sword)

Replace the pysolarcloud OAuth2 dependency with a built-in API client
that uses the iSolarCloud OpenAPI v1 endpoints (login, getPowerStationList,
getDeviceList, getDeviceRealTimeData) with username/password authentication.

Key changes:
- Add isolarcloud_api.py: self-contained async API client for v1 endpoints
- Simplify config_flow to single-step (credentials + login validation)
- Remove OAuth callback view and HTTP dependency from __init__.py
- Remove pysolarcloud from manifest.json requirements
- Update all tests (45 passing)

https://claude.ai/code/session_01XqnmcD4knwew4rrePfen9D
- manifest.json name: "Sungrow iSolarCloud (API v1)"
- Config flow title: "Connect to iSolarCloud (API v1)"
- Entry title: "Sungrow v1 <username>"

https://claude.ai/code/session_01XqnmcD4knwew4rrePfen9D
- Rewrite README to document v1 authentication flow (username/password)
- Document API endpoints used (login, getPowerStationList, etc.)
- Update setup instructions for V1 non-OAuth app creation
- Update .env.example with username/password fields

https://claude.ai/code/session_01XqnmcD4knwew4rrePfen9D
Rename the integration domain and directory to avoid conflicts with the
original sungrow integration.

https://claude.ai/code/session_01XqnmcD4knwew4rrePfen9D
feat: migrate from iSolarCloud API v2 (OAuth) to API v1 (username/pas…
The getPowerStationList and getDeviceList endpoints require curPage and
size parameters. Without them the API returns er_missing_parameter.

https://claude.ai/code/session_01XqnmcD4knwew4rrePfen9D
@pirvu pirvu closed this Mar 7, 2026
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