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.

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

FEAT: Ability to reorder & ability to hide Cameras in UI #2981

Merged

Conversation

NickM-27
Copy link
Sponsor Collaborator

@NickM-27 NickM-27 commented Mar 17, 2022

Implementation of #2782
Implementation of #2837

• Add UI config (for other modifications to the UI in the future)
• Ability to hide cameras from dashboard
• Add order to UI config
• Sort cameras by order. The default is 0 for every camera so the current order is maintained in the default case.
• Ability to hide cameras from birdseye specifically.

@NickM-27
Copy link
Sponsor Collaborator Author

@blakeblackshear Is there more needed on this PR or waiting to be closer to the first beta or later before merging?
Not intending to rush the merge or anything, just hoping to better understand the frigate contribution process. Also thanks for taking the time to review PRs, I've learned a lot through your comments.

@blakeblackshear
Copy link
Owner

I just haven't had time to review outstanding PRs. I am in the middle of working to integrate Frigate+ and will review when I come up for air.

@NickM-27
Copy link
Sponsor Collaborator Author

I just haven't had time to review outstanding PRs. I am in the middle of working to integrate Frigate+ and will review when I come up for air.

Totally understand, I'm sure that's a lot of work to do. Thank you for taking the time to reply 😅

@blakeblackshear blakeblackshear merged commit a5016af into blakeblackshear:release-0.11.0 Apr 15, 2022
@NickM-27 NickM-27 deleted the reorder-hide-cameras branch April 15, 2022 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants