-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
fix: add headers in fabric image component #45415
fix: add headers in fabric image component #45415
Conversation
@cipolleschi Checks are failing . I am not sure why. |
@deepanshushuklad11 main is currently not stable. Once it stabilises, rebase off that commit.
|
d943572
to
3277bd9
Compare
Base commit: d2773f0 |
Thanks @blakef |
@blakef has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
This pull request was successfully merged by @deepanshushuklad11 in 50f7892 When will my fix make it into a release? | How to file a pick request? |
FIXES 45404
Summary:
sending headers from Image component not working in new arch , implementation was missing
<Image source={{ uri: "http://localhost:3000/image", headers: { "test-header": 'test', "hello":"tested" } }} style={{ width: 300, height: 300, }} />
Changelog:
[IOS] [ADDED]- sending missing headers field with Image component in fabric
Test Plan:
Tested
Attaching the below video to show how headers are getting received on server from Image component running in new arch
Screen.Recording.2024-07-12.at.8.15.37.PM.1.1.mov