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.

Sign up for GitHub

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

Jump to bottom

Clarify software support docs for UniFi Protect #22745

Merged
merged 8 commits into from
May 14, 2022
Merged

Clarify software support docs for UniFi Protect #22745

merged 8 commits into from
May 14, 2022

Conversation

AngellusMortis
Copy link
Contributor

@AngellusMortis AngellusMortis commented May 14, 2022 •
edited by bdraco
Loading

Proposed change

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.

@probot-home-assistant probot-home-assistant bot added the current This PR goes into the current branch label May 14, 2022
bdraco
bdraco previously approved these changes May 14, 2022
View reviewed changes
Copy link
Member

@bdraco bdraco left a comment

Choose a reason for hiding this comment

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

Thanks @AngellusMortis

Hopefully this will stem the tide of issue reports

Christopher Bailey and others added 2 commits May 13, 2022 20:33
@bdraco
Update source/_integrations/unifiprotect.markdown
116b3ff 
Co-authored-by: J. Nick Koston <nick@koston.org>
Remove comma
ad0663b
@AngellusMortis

This comment was marked as abuse.

Sign in to view
@bdraco bdraco merged commit 9087510 into home-assistant:current May 14, 2022
@AngellusMortis AngellusMortis deleted the update-ufp-support branch May 14, 2022 01:03
@stivenpl79
Copy link

stivenpl79 commented May 14, 2022 •
edited
Loading

Doesn't work for me.
When i put a ssh and install 2.0.0 bet 5 a get a information:

"E: Version '2.0.0-beta.5' for 'unifi-protect' was not found"

I uninstall protect application on my UDM PRO, disable Early Access and i install old version 1.21.6
And i Home Assistant when i remove my integration and install i have the same error"
"ValueError: 'package' is not a valid SmartDetectObjectType"

@stivenpl79
Copy link

The command line ssh than works for me is:

apt-get install --reinstall --allow-downgrades unifi-protect=2.0.0~beta.5 -y

But this not solve the problem. But i still have error in integration with unifi-protect :(

@github-actions github-actions bot locked and limited conversation to collaborators May 15, 2022
frenck
frenck reviewed May 15, 2022
View reviewed changes
Copy link
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.

Please address these comments in a new PR @AngellusMortis

source/_integrations/unifiprotect.markdown

#### Downgrading UniFi Protect

In the event you accidentally upgrade to an Early Access version of UniFi Protect and it breaks your UniFi Protect Home Assistant integration, you can access your <a href="https://help.ui.com/hc/en-us/articles/204909374#h_01F8G1NSFWE9GWXMT977VQEP8V" target="_blank" rel="noopener">UniFi OS Console via SSH</a> and then do the following:
Copy link
Member

Choose a reason for hiding this comment

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

Please do not use HTML, use Markdown instead.

Copy link
Member

Choose a reason for hiding this comment

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

🤦 I should have caught that. 😦

This comment was marked as abuse.

Sign in to view

Copy link
Member

@frenck frenck May 15, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

Please adjust it to use Markdown as requested, this is not a discussion but a requirement. Would appreciate it being adjusted. Thanks.

source/_integrations/unifiprotect.markdown

You can replace `2.0.0-beta.5` with whatever version of UniFi Protect you want to downgrade to.

**Note**: if you want to downgrade to another Early Access version, you must have <a href="https://help.ui.com/hc/en-us/articles/115012240067-UniFi-How-to-enable-remote-access" target="_blank" rel="noopener">Remote Access enabled</a> and have the Early Access release channel enabled.
Copy link
Member

Choose a reason for hiding this comment

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

Please remove the use of HTML, use Markdown instead.

source/_integrations/unifiprotect.markdown Show resolved Hide resolved
@home-assistant home-assistant unlocked this conversation May 15, 2022
@github-actions github-actions bot locked and limited conversation to collaborators May 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@frenck frenck frenck left review comments

@bdraco bdraco bdraco approved these changes

Assignees
No one assigned
Labels
cla-signed current This PR goes into the current branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

UniFi update Protect 2.0.0-beta.7 crash the integration
5 participants
@AngellusMortis @stivenpl79 @frenck @bdraco @homeassistant