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

Recordings Fixes #13005

Merged
merged 10 commits into from
Aug 12, 2024
Merged

Recordings Fixes #13005

merged 10 commits into from
Aug 12, 2024

Conversation

NickM-27
Copy link
Sponsor Collaborator

  • stop showing activity indicator if recordings load with 0 relevant recordings
  • fix case where refreshing with bad date selected led to crash / unusable UI

Copy link

netlify bot commented Aug 12, 2024

Deploy Preview for frigate-docs canceled.

Name Link
🔨 Latest commit 2266099
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/66ba6c4d758a9500083e4773

@NickM-27 NickM-27 merged commit 2814829 into dev Aug 12, 2024
13 checks passed
@NickM-27 NickM-27 deleted the fix-recording branch August 12, 2024 20:30
sandnabba pushed a commit to sandnabba/frigate that referenced this pull request Aug 14, 2024
* If recordings don't exist mark as no recordings

* Fix reloading recordings failing

* Fix mark items not clearing selected

* Cleanup

* Default to last full hour when error occurs

* Remove check

* Cleanup

* Handle empty recordings list case

* Ensure that the start time is within the time range

* Catch other reset cases
NickM-27 added a commit that referenced this pull request Aug 30, 2024
* If recordings don't exist mark as no recordings

* Fix reloading recordings failing

* Fix mark items not clearing selected

* Cleanup

* Default to last full hour when error occurs

* Remove check

* Cleanup

* Handle empty recordings list case

* Ensure that the start time is within the time range

* Catch other reset cases
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.

2 participants