Fix HEIC image processing context loss with proper error handling#83675
Draft
OlimpiaZurek wants to merge 2 commits intoExpensify:mainfrom
Draft
Fix HEIC image processing context loss with proper error handling#83675OlimpiaZurek wants to merge 2 commits intoExpensify:mainfrom
OlimpiaZurek wants to merge 2 commits intoExpensify:mainfrom
Conversation
Added eslint-disable comments for deprecated manipulateAsync API usage. This is a pre-existing deprecation that should be addressed separately from the HEIC context loss fix. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Codecov Report❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Details
Fixed HEIC image processing context loss issues by adding proper error handling and fallback processing. The changes add robust error handling to prevent crashes when HEIC conversion fails due to context loss, ensuring images can still be processed using fallback methods while providing appropriate logging for debugging.
Fixed Issues
$ #81702
PROPOSAL:
Tests
Offline tests
N/A
QA Steps
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand notonIconClick)myBool && <MyComponent />instead ofmyString && <MyComponent />src/languages/*files and using the translation methodSTYLE.md) were followedScrollViewas the outermost component instead of theView(N/A)useEffectis used, I verified the dependency list is correct or the effect is only executed once (N/A)Avataris modified, I verified thatAvataris working as expected in all cases)Teststeps (N/A)Screenshots/Videos
[No QA] - Internal error handling improvement only, no user-visible changes
🤖 Generated with Claude Code