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

Features/machine update #238

Merged
merged 7 commits into from
Dec 9, 2023
Merged

Features/machine update #238

merged 7 commits into from
Dec 9, 2023

Conversation

cashew22
Copy link
Collaborator

@cashew22 cashew22 commented Dec 4, 2023

This PR add new sensor as request in issue #179 .

image
image

I avoid on purpose adding buttons to trigger the update, I think it's a bit out of scope of the integration.

@cashew22
Copy link
Collaborator Author

cashew22 commented Dec 5, 2023

  • Added a binary sensor that show if an update is available
  • Hide the new sensors by default

Copy link
Owner

@marcolivierarsenault marcolivierarsenault left a comment

Choose a reason for hiding this comment

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

ataboy

Eric Tremblay and others added 6 commits December 9, 2023 12:31
Those entities mimic what you can see in the mainsail UI.
When a new version is available it display:

{version} > {remote_version}

and when the version is up to date, it only display:

{version}

The "system" component is a bit different, it display the number
of packages that need an update
It does not indicate which update is available, just if something can be
updated
@cashew22 cashew22 merged commit 6b427c4 into main Dec 9, 2023
6 checks passed
@cashew22 cashew22 deleted the features/machine-update branch December 9, 2023 17:44
@NonaSuomy
Copy link

Great job! Thank you!

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.

3 participants