Skip to content

Add Update Entity#49

Open
SteveMicroNova wants to merge 4 commits into
mainfrom
AmpliPiUpdate
Open

Add Update Entity#49
SteveMicroNova wants to merge 4 commits into
mainfrom
AmpliPiUpdate

Conversation

@SteveMicroNova

@SteveMicroNova SteveMicroNova commented Jun 12, 2025

Copy link
Copy Markdown

image

image

Adds the ability to notice that the underlying amplipi system has an update as well as install it from home assistant
Note that this suffers from the same issues that the updater itself does, namely failing for unknown reasons and needing to simply hit "update" again

@SteveMicroNova SteveMicroNova marked this pull request as draft June 16, 2025 18:51
@SteveMicroNova SteveMicroNova changed the title Add notification for available updates Add Update Entity Jun 18, 2025
@SteveMicroNova SteveMicroNova requested a review from linknum23 June 23, 2025 18:42
@SteveMicroNova SteveMicroNova marked this pull request as ready for review June 23, 2025 18:42
@SteveMicroNova

Copy link
Copy Markdown
Author

I am now declaring this not a draft, though if we decide to support the full gamut of updates instead of just the latest update we can set it back to draft
A lot of the groundwork is there, I just haven't finished it (nor do I currently see the need)

Comment on lines +18 to +28
class AmpliPiDataSchema(BaseModel):
status: Status
latest: dict
releases: List[dict]

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

A lot of the changes in this PR are just downstream effects of needing to add the github info to the data_coordinator's data stream

@SteveMicroNova SteveMicroNova force-pushed the AddStreamEntities branch 2 times, most recently from 1c8ce8e to ccb44f2 Compare June 25, 2025 18:52
@SteveMicroNova SteveMicroNova force-pushed the AmpliPiUpdate branch 2 times, most recently from f98c24e to f68b0a7 Compare June 25, 2025 19:33
@SteveMicroNova

Copy link
Copy Markdown
Author

This needs to be rebased onto #64 and fixed up a bit before review due to them having conflicting changes that #64 does better

@SteveMicroNova SteveMicroNova marked this pull request as draft June 30, 2025 21:32
…tor and models.py

Move supported features to the parent class

Fix errors caused by moving supported_features property to parent class

Create helper functions to handle the rca-source relationship

fill stream extra_attributes with potentially useful data

Move helper functions from class to global context

Merge AmpliPi client object with AmpliPiCoordinator to get even faster polling
Base automatically changed from AddStreamEntities to main July 1, 2025 16:29
@SteveMicroNova SteveMicroNova changed the base branch from main to ClassRebuild July 1, 2025 17:35
@SteveMicroNova SteveMicroNova marked this pull request as ready for review July 1, 2025 18:27
… rename the _client attribute to _data_client for the same reason
@SteveMicroNova SteveMicroNova force-pushed the AmpliPiUpdate branch 2 times, most recently from 323906d to 63b1592 Compare July 1, 2025 19:55
Base automatically changed from ClassRebuild to main July 15, 2025 20:22
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.

1 participant