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

[$250] Attachement - Console log error appears when sending an attachment #46060

Closed
1 of 6 tasks
lanitochka17 opened this issue Jul 23, 2024 · 23 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 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 retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause

Comments

@lanitochka17
Copy link

lanitochka17 commented Jul 23, 2024

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: 9.0.11-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: N/A
Issue reported by: Applause - Internal Team

Issue found when executing PR #45366

Action Performed:

  1. Go to the staging app
  2. Open Console
  3. Copy any image to your clipboard
  4. Open any report
  5. Paste the image to the composer input by Ctrl+V
  6. Press the 'Enter' key to send the message
  7. Observe the console tab while the action is being performed

Expected Result:

No console error will show up

Actual Result:

Console error shows up

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6550690_1721751064780.2024-07-23_18_46_19.mp4

logs (2).txt

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0108693be973624c6a
  • Upwork Job ID: 1816579481007178354
  • Last Price Increase: 2024-08-08
@lanitochka17 lanitochka17 added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jul 23, 2024
Copy link

melvin-bot bot commented Jul 23, 2024

Triggered auto assignment to @johncschuster (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@lanitochka17
Copy link
Author

@johncschuster FYI 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

@lanitochka17
Copy link
Author

We think that this bug might be related to #vip-vsp

@johncschuster
Copy link
Contributor

I have reproduced this

2024-07-25_15-55-50.mp4

@melvin-bot melvin-bot bot removed the Overdue label Jul 25, 2024
@johncschuster johncschuster added the External Added to denote the issue can be worked on by a contributor label Jul 25, 2024
Copy link

melvin-bot bot commented Jul 25, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0108693be973624c6a

@melvin-bot melvin-bot bot changed the title Attachement - Console log error appears when sending an attachment [$250] Attachement - Console log error appears when sending an attachment Jul 25, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jul 25, 2024
Copy link

melvin-bot bot commented Jul 25, 2024

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

@trjExpensify
Copy link
Contributor

This is a chat attachment bug, better suited for #vip-vsb.

Copy link

melvin-bot bot commented Jul 29, 2024

@johncschuster, @Pujan92 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot melvin-bot bot added the Overdue label Jul 29, 2024
@johncschuster
Copy link
Contributor

@Pujan92 can you take a look? It's not clear to me if this can be resolved by an external contributor or if it has to be resolved internally, and that will help me decide if we're going to prioritize this.

@Pujan92
Copy link
Contributor

Pujan92 commented Jul 30, 2024

@johncschuster We can let it open for the external contributors as I see some error in Onyx saving.

Screenshot 2024-07-30 at 16 34 13

@melvin-bot melvin-bot bot removed the Overdue label Jul 30, 2024
Copy link

melvin-bot bot commented Aug 1, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@johncschuster
Copy link
Contributor

@Pujan92 I'm confused by your last comment. Can you clarify that for me? Do you feel this can be resolved by an external contributor, or do you feel we will need to take this internal?

@Pujan92
Copy link
Contributor

Pujan92 commented Aug 2, 2024

@johncschuster It seems more like a front-end issue only, so we can keep this open for external contributors.

@melvin-bot melvin-bot bot removed the Overdue label Aug 2, 2024
@ardent-dev
Copy link

I'm unable to reproduce this issue on staging/prod - no error is produced in the console when adding an attachment. Can you confirm if this is still an issue?

Copy link

melvin-bot bot commented Aug 5, 2024

@johncschuster, @Pujan92 Whoops! This issue is 2 days overdue. Let's get this updated quick!

@melvin-bot melvin-bot bot added the Overdue label Aug 5, 2024
@johncschuster
Copy link
Contributor

It seems more like a front-end issue only, so we can keep this open for external contributors.

Got it! Thanks for clarifying that! In that case, let's see if we can drum up some proposals!

@melvin-bot melvin-bot bot removed the Overdue label Aug 5, 2024
Copy link

melvin-bot bot commented Aug 6, 2024

@johncschuster @Pujan92 this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@Pujan92
Copy link
Contributor

Pujan92 commented Aug 7, 2024

Thanks @ardent-dev.

@johncschuster I am also not able to reproduce this now, can you plz confirm?

Copy link

melvin-bot bot commented Aug 8, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@melvin-bot melvin-bot bot added the Overdue label Aug 9, 2024
@johncschuster johncschuster added the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Aug 9, 2024
@johncschuster
Copy link
Contributor

@Pujan92

I am also not able to reproduce this now, can you plz confirm?

Sure! I've just asked for a retest. If the issue isn't reproducible, let's close the issue.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Aug 9, 2024
@kavimuru
Copy link

Not reproducible anymore.

2024-08-12.08.11.13.mp4

Copy link

melvin-bot bot commented Aug 13, 2024

@johncschuster, @Pujan92 Whoops! This issue is 2 days overdue. Let's get this updated quick!

@johncschuster
Copy link
Contributor

Ok! I'm going to go ahead and close this one then. Thanks for the retest!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 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 retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause
Projects
No open projects
Archived in project
Development

No branches or pull requests

6 participants