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 - Add image from gallery doesn't show the preview before uploading #3385

Closed
isagoico opened this issue Jun 5, 2021 · 5 comments · Fixed by #3408
Closed

Android - Add image from gallery doesn't show the preview before uploading #3385

isagoico opened this issue Jun 5, 2021 · 5 comments · Fixed by #3408
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@isagoico
Copy link

isagoico commented Jun 5, 2021

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Expected Result:

Preview should be displayed before uploading the picture

Actual Result:

No image is displayed, only the "chat_attachment" screen. After you click upload, the image is uploaded without issues.

Action Performed:

  1. Open e.cash app in Android
  2. Navigate to a conversation
  3. Click on the attachment button
  4. Click on choose from gallery
  5. Select any picture

Workaround:

No need, image is uploaded without issues.

Platform:

Where is this issue occurring?

Web
iOS
Android ✔️
Desktop App
Mobile Web

Version Number: 1.0.63-0

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:

WhatsApp.Video.2021-06-04.at.8.51.58.PM.mp4

Expensify/Expensify Issue URL:

View all open jobs on Upwork

@MelvinBot
Copy link

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

@isagoico
Copy link
Author

isagoico commented Jun 7, 2021

Issue reproducible today during KI retests

@sketchydroide
Copy link
Contributor

This seems like something we should work on, maybe we can open this to externals.

@sketchydroide sketchydroide added the External Added to denote the issue can be worked on by a contributor label Jun 7, 2021
@MelvinBot
Copy link

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

@sketchydroide sketchydroide added Weekly KSv2 and removed Daily KSv2 labels Jun 7, 2021
@sketchydroide sketchydroide removed their assignment Jun 7, 2021
@dklymenk
Copy link
Contributor

dklymenk commented Jun 7, 2021

This will be fixed by
#3408

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 Weekly KSv2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants