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.

Sign up for GitHub

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

Jump to bottom

Initial refactor of integration based on jacobdonenfeld/homeassistant-aerogarden #2

Merged
merged 9 commits into from
Aug 22, 2023
Merged

Initial refactor of integration based on jacobdonenfeld/homeassistant-aerogarden #2

merged 9 commits into from
Aug 22, 2023

Conversation

dalinicus
Copy link
Owner

  • Break apart init.py
    • Create dedicated client for aerogarden api
    • Create dedicated data orchestrator
  • Implement code flow configuration
  • Redesign sensors
    • Make unique_ids more unique by using field instead of label to assemble the id
    • Assign sensors device classes
    • Use built in units where applicable
    • Convert integer based water level to enum based
    • Temporarily convert Light sensor to Binary Sensor pending refactor
  • Convert operations to async where applicable
  • Add MIT License
  • Temporarily disable workflows pending refactor

@dalinicus dalinicus merged commit 9c48b3e into main Aug 22, 2023
@dalinicus dalinicus deleted the feature/v1-refactor branch August 22, 2023 16:14
@dalinicus dalinicus mentioned this pull request Aug 22, 2023
Closed
1 task
@dalinicus
Copy link
Owner Author

Fixes jacobdonenfeld/homeassistant-aerogarden#7

@dalinicus dalinicus mentioned this pull request Sep 4, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dalinicus