-
Notifications
You must be signed in to change notification settings - Fork 24.2k
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
[iOS] Use mainBundle in RCTConvert to resolve asset paths #3889
Conversation
By analyzing the blame information on this pull request, we identified @javache, @nicklockwood and @sahrens to be potential reviewers. |
@facebook-github-bot shipit |
Thanks for importing. If you are an FB employee go to https://our.intern.facebook.com/intern/opensource/github/pull_request/505905282925767/int_phab to review. |
This broke some of our internal tests which were relying on assets located in the test bundle. I'm going to need to solve that before I can land this. The fix itself is correct though - we'll find another solution for the tests. |
Sure thing, thanks for letting me know! |
f8df384
Summary: See facebook#3888 for why this is necessary. Essentially, `[NSBundle mainBundle]` loads the file path for the target app which is the only way to reference images. cc javache nicklockwood Closes facebook#3889 Reviewed By: svcscm Differential Revision: D2615580 Pulled By: nicklockwood fb-gh-sync-id: d06ce0987dde666b06bb5a7edf609ed45f325d2c
Summary: See facebook#3888 for why this is necessary. Essentially, `[NSBundle mainBundle]` loads the file path for the target app which is the only way to reference images. cc javache nicklockwood Closes facebook#3889 Reviewed By: svcscm Differential Revision: D2615580 Pulled By: nicklockwood fb-gh-sync-id: d06ce0987dde666b06bb5a7edf609ed45f325d2c
See #3888 for why this is necessary. Essentially,
[NSBundle mainBundle]
loads the file path for the target app which is the only way to reference images.Test plan: Make sure unit tests pass!
cc @javache @nicklockwood