-
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
[ON HOLD #47203] [HIGH] [$500] mWeb - Composer-App crashes/freezes when pasting copied image inside the compose box #36909
Comments
Triggered auto assignment to @CortneyOfstad ( |
@CortneyOfstad I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors. |
Thanks @kbecciv! I've been diving into this and I think that this could be handled by an external contrubutor 👍 |
Job added to Upwork: https://www.upwork.com/jobs/~0131dabafc5db30095 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eh2077 ( |
@eh2077 Please let me know if you have any concerns about this potentially being a BE Issue, and we can get that tags switched. From what I could see, it appears not to be, but would love fresh eyes just in case 👍 |
@CortneyOfstad Yeah, this is most likely an issue from frontend. |
Posted in VIP-VSB here |
@CortneyOfstad, @eh2077 Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Going to reach out to CallStack to see if someone can take this on 👍 |
Post to CS here |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Some information if it help. I can't reproduce the bug. In the gallery app(which I downloaded from google play), there is no 'copy to clipboard' option, I tried 3~4 apps. Finally I noticed in app swiper we can copy image to clipboard. When I copy the image to composer of expensify, everything works fine. I am a ios user and rarely use android device by the way. |
@badeggg Thanks for looking into this! I think the photo app is Samsung Gallery. You can search keyword |
Discussed with @trjExpensify too, for now I think we can close this, its not that often used feature and we can spend our time in better manner now so closing and we can reopen once we ready to implement this |
Issue is still reproducible on the latest build 9.0.20-2 Screen_Recording_20240814_154411_New.Expensify.mp4 |
Did #41239 solve this issue? |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Still waiting for #47203 to hit production, so going to put this on-hold |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Okay, botify is sick, so the move to production was done manually without an update to the PR. It went into Production 2 days, ago so the 7-day wait period is over on Aug. 26! |
Issue not reproducible during KI retests. (First week) |
The hold period for production has been completed so this can be closed! |
Let's keep this one open, it seems to be reproducible on Samsung devices only. cc @ahmedGaber93 can you please confirm this is still reproducible from your end |
It reproduced on production, and I will retest the latest main branch today. |
@s77rt The feature itself is not working with me on android (on both emulator and real device) after RN upgrading to 0.75.2. 20240829031128673.mp4 |
@ahmedGaber93 Can you try remove the |
@s77rt Yes, I already did that before the build. And I think the patch is applied correctly because it works fine on iOS |
Indeed, looks like something broke it. Will take a look asap |
@ahmedGaber93 The patch fix #48370 |
@s77rt Yes, it still reproduced on real Samsung device |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.43.0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4331256
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
App should function normally and paste the image
Actual Result:
App freezes and then crashes eventually
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6385684_1708433272308.App_crash_pasting_image.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: