Skip to content
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

Android/Moto G7 - Attaching picture directly from camera will crash the app #4853

Closed
kavimuru opened this issue Aug 26, 2021 · 21 comments
Closed
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Weekly KSv2

Comments

@kavimuru
Copy link

kavimuru commented Aug 26, 2021

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Open app and login
  2. Open any chat
  3. Tap on + > Add Attachment > Take Photo
  4. Take a picture and confirm

Expected Result:

User is able to attach taken picture to the conversation

Actual Result:

Accepting the picture on camera will crash the app

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • Android ✔️

Version Number:
1.0.88
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Issue was only reproducible in Motorola Moto G7 Play - Android 10, we tried on other devices and the crash was not reproducible.

Crash logs

Bug5210378_crash-fresh.mp4

View all open jobs on GitHub

@MelvinBot
Copy link

Triggered auto assignment to @thienlnam (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@thienlnam thienlnam added the External Added to denote the issue can be worked on by a contributor label Aug 26, 2021
@MelvinBot
Copy link

Triggered auto assignment to @NicMendonca (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@NicMendonca
Copy link
Contributor

@thienlnam just confirming if you were able to reproduce this? (I don't have an Android)

@thienlnam
Copy link
Contributor

I also don't have an android and didn't try to reproduce - how about this, @kavimuru, can you let us know if this comes up in the next retest and if it does we'll continue to create an upwork job for it?

@parasharrajat
Copy link
Member

It's really hard to test. The best thing here is to see the crash report sent to the backend for this device.

@isagoico
Copy link

isagoico commented Aug 30, 2021

Issue not reproducible during KI retests (It was tested on other devices). If needed I can ask the tester to record the crash logs again.

@NicMendonca
Copy link
Contributor

@kavimuru as per @isagoico last message ☝️ Can you please see if you can reproduce/ record the crash logs again?

@MelvinBot MelvinBot added Overdue and removed Overdue labels Aug 30, 2021
@kavimuru
Copy link
Author

kavimuru commented Sep 1, 2021

@NicMendonca Tester is able to reproduce 1 out of 10 times in the latest build (1.0.91.0)
Video and logs are attached.

Screen.Recording.9-1-2021.3-56-19.PM.mp4

bugreport-channel-QPYS30.52-22-5-2021-09-01-15-56-24.zip

@thienlnam
Copy link
Contributor

@NicMendonca We should still just make this an external issue and then just have a contributor investigate

@MelvinBot MelvinBot added Overdue Help Wanted Apply this label when an issue is open to proposals by contributors labels Sep 1, 2021
@MelvinBot
Copy link

Triggered auto assignment to @AndrewGable (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@NicMendonca NicMendonca added Weekly KSv2 and removed Daily KSv2 labels Sep 1, 2021
@MelvinBot MelvinBot removed the Overdue label Sep 1, 2021
@NicMendonca
Copy link
Contributor

Upwork job posting here.

@AndrewGable
Copy link
Contributor

Probably just have to update a library to fix this one I would imagine, since QA can verify the fix pretty easily the proposal should be pretty straightforward.

@isagoico
Copy link

isagoico commented Sep 5, 2021

Issue not reproducible during KI retests. Also, tester who raised the issue confirmed he was unable to reproduce this anymore.

@mananjadhav
Copy link
Collaborator

I had an issue attaching an image from the OnePlus 6. It didn't crash the app, it just hangs up.

@AndrewGable
Copy link
Contributor

Only on the OnePlus 6?

@mananjadhav
Copy link
Collaborator

I only have OnePlus to test in Android.

@AndrewGable
Copy link
Contributor

Can you try attaching a smaller image and seeing if it ever uploads (e.g. after 15 or 30 seconds)? I know those phones take large photos and can take a while to upload.

@mananjadhav
Copy link
Collaborator

Gallery upload worked fine. Camera it didn’t and I must’ve waited for about 2-3 mins. I uploaded a 2.18 mb image from iOS during the same test

@AndrewGable
Copy link
Contributor

@mananjadhav - Do you want to explore this issue and propose a fix once we have more information?

@mananjadhav
Copy link
Collaborator

@AndrewGable I am not sure if I raised a false alarm or that it worked earlier and now stopped working? I downloaded the app from Playstore and it seems to be working.

This was the app running with USB connected. If you look at the video from 00:30s you'll see the behavior that I experienced.

android-recorder.mp4

@AndrewGable
Copy link
Contributor

Weird for sure. Let's close this issue for now and reopen if we can reproduce reliably.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Weekly KSv2
Projects
None yet
Development

No branches or pull requests

8 participants