Skip to content

Conversation

@gloriacai01
Copy link
Member

@gloriacai01 gloriacai01 commented Jan 2, 2026

This pr adds the check for the error that we added in app when the include_binary is true and the file being requested too large.

@viambot viambot added the safe to test This pull request is marked safe to test from a trusted zone label Jan 2, 2026
@gloriacai01 gloriacai01 requested a review from n0nick January 2, 2026 21:26
Copy link
Member

@n0nick n0nick left a comment

Choose a reason for hiding this comment

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

lgtm with a comment

Co-authored-by: Sagie Maoz <[email protected]>
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Jan 5, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Jan 5, 2026
@gloriacai01 gloriacai01 merged commit 3a0c714 into viamrobotics:main Jan 6, 2026
19 checks passed
@gloriacai01 gloriacai01 deleted the app14607 branch January 6, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test This pull request is marked safe to test from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants