-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[CP Staging] Revert "[New architecture] Replace @oguzhnatly/react-native-image-manipulator
with expo-image-manipulator
"
#38024
Conversation
…nipulator` with `expo-image-manipulator`"
skipping checklists for a straight revert |
Reviewer Checklist
Screenshots/VideosAndroid: NativeAndroid: mWeb ChromeiOS: NativeiOS: mWeb SafariMacOS: Chrome / SafariMacOS: Desktop |
@roryabraham looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
skipped checklist for automatic straight revert |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
…manipulator Revert "[New architecture] Replace `@oguzhnatly/react-native-image-manipulator` with `expo-image-manipulator`" (cherry picked from commit f04038f)
🚀 Cherry-picked to staging by https://github.com/roryabraham in version: 1.4.49-4 🚀
@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes. |
@oguzhnatly/react-native-image-manipulator
with expo-image-manipulator
"@oguzhnatly/react-native-image-manipulator
with expo-image-manipulator
"
🚀 Deployed to production by https://github.com/luacmartins in version: 1.4.50-5 🚀
|
Reverts #36019 - straight revert
Fixes: #37963