-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Attachment-App crashes when scrubbing through an mp4 file on attachment upload preview #36905
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
Comments
Unable to auto-create job on Upwork. The BZ team member should create it manually for this issue. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra ( |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Triggered auto assignment to @rlinoz ( |
We think that this bug might be related to #vip-vsp |
Related to #30829 |
Hey! I'm not sure if I should write proposal since the fix is quite straight forward and we need to fix it as quick as it's possible. Cc. @mountiny |
Maybe a tiny explanation on PR itself will work. |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
@kowczarz no need for proposals on this |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
Removing blocker label since the PR was CP-d to staging. |
Issue not reproducible during KI retests. (First week) |
Triggered auto assignment to @isabelastisser ( |
This is actually done and @shubham1206agra was the C+ reviewer. |
Thanks for the heads up @rlinoz! @shubham1206agra, I've sent you an offer in Upwork now, please accept it and I will process the payment. Thanks! All set. |
@isabelastisser Why the offer of $250? Is there a reason for this? |
@rlinoz @shubham1206agra, can you please confirm the payment amount for the C+ review here? Is it the standard $500? |
Yeah, it should be the standard rate |
Thanks! I updated the payment amount an d processed it in Upwork. All set. |
Uh oh!
There was an error while loading. Please reload this page.
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 1.4.43-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4331252
Email or phone of affected tester (no customers): [email protected]
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
App should play the video normally
Actual Result:
App crashes
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6385670_1708432555805.Crash_with_video.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: