Skip to content

Update Lightwave to remove need for proxy#25625

Merged
MartinHjelmare merged 1 commit into
home-assistant:nextfrom
ColinRobbins:patch-2
May 24, 2023
Merged

Update Lightwave to remove need for proxy#25625
MartinHjelmare merged 1 commit into
home-assistant:nextfrom
ColinRobbins:patch-2

Conversation

@ColinRobbins
Copy link
Copy Markdown
Contributor

@ColinRobbins ColinRobbins commented Jan 7, 2023

Proposed change

The Lightwave TRV integration previously required a 3rd party proxy to listen for UDP events from the TRVs.
A PR removed the need for the proxy, by building the UDP listener into the integration.
This updates the documentation to match the PR

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@home-assistant home-assistant Bot added the next This PR goes into the next branch label Jan 7, 2023
@home-assistant home-assistant Bot added the has-parent This PR has a parent PR in another repo label Jan 7, 2023
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

✅ Approved. Can be merged as soon as the parent PR gets merged.

@frenck frenck added the awaits-parent Awaits the merge of an parent PR label Jan 16, 2023
@MartinHjelmare MartinHjelmare added parent-merged The parent PR has been merged already and removed awaits-parent Awaits the merge of an parent PR labels May 24, 2023
@MartinHjelmare MartinHjelmare changed the title Update Lightware to remove need for proxy. Update Lightware to remove need for proxy May 24, 2023
@MartinHjelmare MartinHjelmare changed the title Update Lightware to remove need for proxy Update Lightwave to remove need for proxy May 24, 2023
@MartinHjelmare MartinHjelmare merged commit 75f2244 into home-assistant:next May 24, 2023
@home-assistant home-assistant Bot removed the parent-merged The parent PR has been merged already label May 24, 2023
@github-actions github-actions Bot locked and limited conversation to collaborators May 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants