Skip to content

add agent_dvr#12333

Merged
frenck merged 9 commits intohome-assistant:nextfrom
ispysoftware:next
May 11, 2020
Merged

add agent_dvr#12333
frenck merged 9 commits intohome-assistant:nextfrom
ispysoftware:next

Conversation

@ispysoftware
Copy link
Copy Markdown
Contributor

@ispysoftware ispysoftware commented Mar 11, 2020

adds info for the agent_dvr integration
brands pull request:
home-assistant/brands#50

Proposed change

adding an integration page for our agent_dvr integration

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

  • Link to parent pull request in the codebase:
  • This PR fixes or closes issue:

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.

adds info for the agent_dvr integration
@probot-home-assistant probot-home-assistant Bot added the next This PR goes into the next branch label Mar 11, 2020
@klaasnicolaas klaasnicolaas added the new-integration This PR adds documentation for a new Home Assistant integration label Mar 11, 2020
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.

Thanks for the PR @ispysoftware!

I've reviewed it, and left some comments. Could you please take a look? Thanks! 👍

Comment thread source/_integrations/agent_dvr.markdown Outdated
Comment thread source/_integrations/agent_dvr.markdown Outdated
Comment thread source/_integrations/agent_dvr.markdown Outdated
Comment thread source/_integrations/agent_dvr.markdown Outdated
Comment thread source/_integrations/agent_dvr.markdown Outdated
@frenck frenck added in-progress This PR/Issue is currently being worked on has-parent This PR has a parent PR in another repo labels Mar 11, 2020
@frenck
Copy link
Copy Markdown
Member

frenck commented Mar 11, 2020

Oh please add a link to the parent PR in the openings post of this PR, since it is now unclear with PR in the codebase it belongs to.

Thanks! 👍

ispysoftware and others added 4 commits March 12, 2020 07:37
ispysoftware added a commit to ispysoftware/brands that referenced this pull request Mar 12, 2020
@ispysoftware
Copy link
Copy Markdown
Contributor Author

Thanks @frenck I think i've covered everything - if not please let me know.

@ispysoftware ispysoftware requested a review from frenck March 12, 2020 09:09
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.
⚠️ The ha_release might need an update before merging this one in.

@frenck frenck added awaits-parent Awaits the merge of an parent PR and removed in-progress This PR/Issue is currently being worked on labels Mar 12, 2020
frenck
frenck previously approved these changes Mar 12, 2020
fabaff pushed a commit to home-assistant/brands that referenced this pull request Mar 18, 2020
@frenck frenck added the parent-merged The parent PR has been merged already label May 8, 2020
@frenck frenck self-assigned this May 8, 2020
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.

Thanks for the PR @ispysoftware!

Left some small review comments, could you take a look? Thanks! 👍

Comment thread source/_integrations/agent_dvr.markdown Outdated
Comment thread source/_integrations/agent_dvr.markdown
Comment thread source/_integrations/agent_dvr.markdown
Comment thread source/_integrations/agent_dvr.markdown
Comment thread source/_integrations/agent_dvr.markdown
Comment thread source/_integrations/agent_dvr.markdown Outdated
@frenck frenck added in-progress This PR/Issue is currently being worked on and removed awaits-parent Awaits the merge of an parent PR labels May 9, 2020
@frenck frenck assigned ispysoftware and unassigned frenck May 9, 2020
ispysoftware and others added 2 commits May 11, 2020 08:50
@frenck frenck self-assigned this May 11, 2020
frenck
frenck previously approved these changes May 11, 2020
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.

Thanks, @ispysoftware 👍

@frenck frenck merged commit bf8a68a into home-assistant:next May 11, 2020
@probot-home-assistant probot-home-assistant Bot removed in-progress This PR/Issue is currently being worked on parent-merged The parent PR has been merged already labels May 11, 2020
techcoyangfeng added a commit to techcoyangfeng/brands that referenced this pull request Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants