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.

Sign up for GitHub

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

Jump to bottom

expo-media-library is throwing an error without any useful details #30441

Open
dsemelianov opened this issue Jul 17, 2024 · 2 comments · May be fixed by #30504
Open

expo-media-library is throwing an error without any useful details #30441

dsemelianov opened this issue Jul 17, 2024 · 2 comments · May be fixed by #30504
Assignees
@vonovak
Labels
Issue accepted

Comments

@dsemelianov
Copy link

Minimal reproducible example

N/A

What platform(s) does this occur on?

iOS

Where did you reproduce the issue?

in a standalone app

Summary

I am using expo-media-library in a production iOS app. Some of my users run into this error when deleting photos: "Couldn't remove assets"

Looking at the expo source code, I can see that this error message corresponds with this error:

expo/packages/expo-media-library/ios/MediaLibraryExceptions.swift

Line 69 in 0797cb2

internal class RemoveAssetsException: Exception {

Which is referenced here:

expo/packages/expo-media-library/ios/MediaLibraryModule.swift

Line 210 in 0797cb2

promise.reject(RemoveAssetsException())

Is it possible to get some additional error info along with this message? As it currently stands, there is absolutely no information to go off of in order to debug the issue.

Environment

N/A

Expo Doctor Diagnostics

N/A
@dsemelianov dsemelianov added the needs validation Issue needs to be validated label Jul 17, 2024
@expo-bot expo-bot added needs review Issue is ready to be reviewed by a maintainer and removed needs validation Issue needs to be validated labels Jul 17, 2024
@vonovak
Copy link
Contributor

vonovak commented Jul 18, 2024

Hello and thanks for reporting,
We're going to provide more information about the underlying error.
Thank you 🙂

@vonovak vonovak added Issue accepted and removed needs review Issue is ready to be reviewed by a maintainer labels Jul 18, 2024
@vonovak vonovak self-assigned this Jul 18, 2024
@expo-bot
Copy link
Collaborator

Thank you for filing this issue!
This comment acknowledges we believe this may be a bug and there’s enough information to investigate it.
However, we can’t promise any sort of timeline for resolution. We prioritize issues based on severity, breadth of impact, and alignment with our roadmap. If you’d like to help move it more quickly, you can continue to investigate it more deeply and/or you can open a pull request that fixes the cause.

@vonovak vonovak linked a pull request Jul 18, 2024 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vonovak vonovak

Labels
Issue accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[media-library]: include more error information in native rejections expo/expo
3 participants
@vonovak @dsemelianov @expo-bot