Skip to content

[HOLD for payment 2022-03-17] [HOLD for payment 2022-03-09] Feature Request: Add image preview placeholder while uploading images/Videos - Reported by @mdneyazahmad #7463

Closed

Description

@mvtglobally

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


Action Performed:

  1. Open app
  2. Go to chat on slower network
  3. Upload image and note the loading

Expected Result:

Image preview should display while uploading images

Actual Result:

No Image preview displayed while uploading images

Workaround:

unknown

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.1.33-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screenshot_2022-01-19-00-00-25-84_4f9154176b47c00da84e32064abf1c48

Expensify/Expensify Issue URL:
Issue reported by: @mdneyazahmad
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1642530762139600

View all open jobs on GitHub

Activity

  1. MelvinBot commented on Jan 31, 2022

    @MelvinBot
    Contributor

    Triggered auto assignment to @JmillsExpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

  2. removed
    AutoAssignerTriageAuto assign issues for triage to an available triage team member
    on Jan 31, 2022
  3. changed the title [-]Feature Request: Add image preview placeholder while uploading images - Reported by @mdneyazahmad[/-] [+]Feature Request: Add image preview placeholder while uploading images/Videos - Reported by @mdneyazahmad[/+] on Jan 31, 2022
  4. JmillsExpensify commented on Feb 1, 2022

    @JmillsExpensify
    Contributor

    I agree that from a best practice perspective, we should be providing a local preview of the receipt on upload. That said, I think we want to be careful on how we implement this, so we'll want to make sure any proposals are discussed here and in Slack. I'll go ahead and create a job in the meantime though!

  5. JmillsExpensify commented on Feb 1, 2022

    @JmillsExpensify
    Contributor
  6. MelvinBot commented on Feb 1, 2022

    @MelvinBot
    Contributor

    Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported)

  7. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Feb 1, 2022
  8. 56 remaining items

  9. added and removed on Mar 10, 2022
  10. changed the title [-][HOLD for payment 2022-03-09] Feature Request: Add image preview placeholder while uploading images/Videos - Reported by @mdneyazahmad[/-] [+][HOLD for payment 2022-03-17] [HOLD for payment 2022-03-09] Feature Request: Add image preview placeholder while uploading images/Videos - Reported by @mdneyazahmad[/+] on Mar 10, 2022
  11. botify commented on Mar 10, 2022

    @botify

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.1.41-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2022-03-17. 🎊

  12. JmillsExpensify commented on Mar 10, 2022

    @JmillsExpensify
    Contributor

    Ah interesting, it looks like Jules and I aligned payment to the first PR, not the second one.

  13. okansahin commented on Mar 10, 2022

    @okansahin
    Contributor

    Ah, may it was my fault. I asked for a regression period from @Julesssss

    I thought it was production deploy but now I see it is staging. I'm sorry about that. I follow the issue to fix if any trouble.

    @Julesssss I am really sorry to make confused you too.

  14. JmillsExpensify commented on Mar 11, 2022

    @JmillsExpensify
    Contributor

    No worries! I'm going to go ahead and update the issue with the updated date so that we don't forget. I'll go ahead and close it out if we get through the regression period and no issues.

  15. JmillsExpensify commented on Mar 23, 2022

    @JmillsExpensify
    Contributor

    Circling back to this issue to formally close us out.

  16. mdneyazahmad commented on Mar 24, 2022

    @mdneyazahmad
    Contributor

    @JmillsExpensify I noticed that the job for this issue on Upwork is not closed. Thank you

  17. JmillsExpensify commented on Mar 25, 2022

    @JmillsExpensify
    Contributor

    Screen Shot 2022-03-25 at 1 02 01 PM

    Sorry can you help clarify?

  18. JmillsExpensify commented on Mar 25, 2022

    @JmillsExpensify
    Contributor

    Oh I think I see what you mean! Looks like I didn't close out your specific contract, so I've done that now. Sorry for missing earlier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionExternalAdded to denote the issue can be worked on by a contributorNewFeatureSomething to build that is a new item.WeeklyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions