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

Add detailed errors for unsupported label uploading to frigate+ #9775

Conversation

Maximo-Guk
Copy link
Contributor

@Maximo-Guk Maximo-Guk commented Feb 10, 2024

I ran into #8961 recently when I was getting started with Frigate+. Some of my uploads were initially getting stuck in this "Uploading..." state. I realized soon after that some of my labels weren't supported by Frigate+

Not sure if I should be targeting dev with this PR given that these are changes to "web-old". Also not sure if this issue has already been resolved in the new 0.14 UI!

Current upload error behaviour:
image

Proposed unsupported label error:
Screenshot 2024-02-10 at 12 16 29 AM

Proposed generic upload error:
Screenshot 2024-02-09 at 11 18 05 PM

Copy link

netlify bot commented Feb 10, 2024

Deploy Preview for frigate-docs ready!

Name Link
🔨 Latest commit 6de82cf
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/65c79019e9b1250008d9368f
😎 Deploy Preview https://deploy-preview-9775--frigate-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Maximo-Guk Maximo-Guk force-pushed the handling-unsupported-label-uploads branch from 56a842d to ce92876 Compare February 10, 2024 04:17
@blakeblackshear
Copy link
Owner

If you can rebase this off of master instead of dev, then I will consider including it in the next minor release.

@Maximo-Guk Maximo-Guk force-pushed the handling-unsupported-label-uploads branch from ce92876 to 6de82cf Compare February 10, 2024 15:02
@Maximo-Guk Maximo-Guk changed the base branch from dev to master February 10, 2024 15:03
@Maximo-Guk
Copy link
Contributor Author

Rebased off of master, and now targetting master

@blakeblackshear blakeblackshear merged commit 86341c3 into blakeblackshear:master Feb 10, 2024
4 checks passed
@Maximo-Guk Maximo-Guk deleted the handling-unsupported-label-uploads branch February 10, 2024 19:51
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

2 participants