Skip to content

Hardware MVP#12773

Merged
bramkragten merged 6 commits intodevfrom
hardware-mvp
May 25, 2022
Merged

Hardware MVP#12773
bramkragten merged 6 commits intodevfrom
hardware-mvp

Conversation

@zsarnett
Copy link
Copy Markdown
Contributor

@zsarnett zsarnett commented May 25, 2022

Breaking change

Moves Reboot and Shutdown to overflow (not really breaking)

Proposed change

CleanShot 2022-05-24 at 23 55 30

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@spacegaier
Copy link
Copy Markdown
Member

Moves Reboot and Shutdown to overflow (not really breaking)

Just curios: Is there a reason for that? Hiding things in sub menus without a real need always introduces extra clicks. On a hardware dedicated page, those buttons seem rather fitting.

@zsarnett
Copy link
Copy Markdown
Contributor Author

Just curios: Is there a reason for that?

The plans for this page will continue to grow and going ahead and placing them here will be the best instead of waiting. Something we should have done before

documentationURL = boardData.url;
imageURL = hardwareBrandsUrl({
domain: "boards",
name: `${boardData.board.manufacturer}_${boardData.board.model}`,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we not just pass manufacturer and model as separate params and let hardwareBrandsUrl handle concatenation?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Im not sure that we can guarentee that manufacturer and model will be the defining points for this. Zwave devices for example. they may have other features to define the name

But I think we can do this for now and pivot

this._OSData = await fetchHassioHassOsInfo(this.hass);
}
if (isComponentLoaded(this.hass, "hassio")) {
this._hostData = await fetchHassioHostInfo(this.hass);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

now host data is not loaded when the hardware integration is loaded?

@bramkragten bramkragten merged commit c4624fa into dev May 25, 2022
@bramkragten bramkragten deleted the hardware-mvp branch May 25, 2022 15:11
@github-actions github-actions bot locked and limited conversation to collaborators May 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants