Skip to content

Add feedback dialog for ZHA device reconfiguration#8742

Merged
bramkragten merged 3 commits intohome-assistant:devfrom
dmulcahey:dm/zha-reconfigure-feedback
Mar 31, 2021
Merged

Add feedback dialog for ZHA device reconfiguration#8742
bramkragten merged 3 commits intohome-assistant:devfrom
dmulcahey:dm/zha-reconfigure-feedback

Conversation

@dmulcahey
Copy link
Copy Markdown
Contributor

Proposed change

This PR adds support for visual feedback for ZHA device reconfiguration.

REQUIRES home-assistant/core#48447

reconfigure_device.mov

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:

@dmulcahey
Copy link
Copy Markdown
Contributor Author

@bramkragten when you have a sec... I am struggling w/ something silly with this. The dialog works and shows for the first invocation. I have to refresh the page to make it work again after it is dismissed... What am I missing?

@dmulcahey dmulcahey marked this pull request as ready for review March 30, 2021 11:08
Comment on lines +67 to +73
<paper-textarea
readonly
max-rows="10"
class="log"
value="${this._formattedEvents}"
>
</paper-textarea>
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.

Let's put these technical details inside an expansion-panel

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.

that's all this dialog is right now... I can do that once I devise a way to show this better graphically?

@dmulcahey
Copy link
Copy Markdown
Contributor Author

@bramkragten backend PR merged

@bramkragten bramkragten merged commit 2c886d7 into home-assistant:dev Mar 31, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 1, 2021
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.

3 participants