Skip to content

Update Image Picker Flow#1130

Merged
kevinherdez merged 1 commit intoNWACus:mainfrom
kevinherdez:kevinherdez/imagePickerUpdates
Mar 12, 2026
Merged

Update Image Picker Flow#1130
kevinherdez merged 1 commit intoNWACus:mainfrom
kevinherdez:kevinherdez/imagePickerUpdates

Conversation

@kevinherdez
Copy link
Copy Markdown
Collaborator

This PR remove the deprecated call manipulateAsync in uploadImage and changes the ImagePicker.UIImagePickerPreferredAssetRepresentationMode from Compatible to Current

There's an issue we've been seeing from a couple of forecasters where images fail to load from the ImagePicker with an unexpected error. I had Claude help investigate and one of the potential reasons it came up with is that the representation mode Compatible transcodes the image to the most compatible version. According to Claude, this can potentially fail if the image isn't downloaded on the device which would happen, if the image is backed up to iCloud. By setting it to .Current, the picker will just give us the image which we then transcode to a jpg in uploadImage anyway.

I tested this on iOS and Android (mainly to make sure the the uploadImage flow still worked), and with both .heic and .jpg images.

I do not have images backed up to iCloud and doing this is an all or nothing approach so I was not able to test the suspected failing use case

@rustynwac
Copy link
Copy Markdown
Contributor

Dallas is indeed backing up to iCloud. Waiting to hear from Chugach

@kevinherdez
Copy link
Copy Markdown
Collaborator Author

Dallas is indeed backing up to iCloud. Waiting to hear from Chugach

I'll go ahead and get this in so at least Dallas can test it

@kevinherdez kevinherdez added this pull request to the merge queue Mar 12, 2026
Merged via the queue into NWACus:main with commit 6a0c76a Mar 12, 2026
5 checks passed
@kevinherdez kevinherdez deleted the kevinherdez/imagePickerUpdates branch March 12, 2026 02:47
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