App icon indicating copy to clipboard operation
App copied to clipboard

[HOLD BE work] Workspace chat expense report show up in the LHN without a GBR or as unread

Open m-natarajan opened this issue 1 year ago • 76 comments

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: Reproducible in staging?: needs reproduction Reproducible in production?: needs reproduction If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: @puneetlath Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1712068571511789

Prerequisite Be on a collect plan where User A is approver and User B is an employee User A has priority mode set to #focus

Action Performed:

(also see reproducible steps below from @eVoloshchak )

  1. User B/employee submits expense
  2. User A opens app and observes either
  • The report from user B doesn't show in LHN
  • The report from user B is not bold and/or does not have 🟢 for Green Brick Road (GBR)
  1. Click on the report

Expected Result:

Report should show in LHN and/or GBR should show up in the LHN for the workspace chat

Actual Result:

Report doesn't show in LHN or GBR is not shown and chat is not bold to indicate it is unread very often, when I click on the workspace chat, then a GBR appears.

Seems like perhaps we aren't properly sending the hasOutstandingChildRequest field when a report is sent via instant submit

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platforms:

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

  • [ ] Android: Native
  • [x] Android: mWeb Chrome
  • [ ] iOS: Native
  • [ ] iOS: mWeb Safari
  • [x] MacOS: Chrome / Safari
  • [ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

image (5)

image (4)

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~010d8f311682125da6
  • Upwork Job ID: 1779939910911811584
  • Last Price Increase: 2024-06-06
Issue OwnerCurrent Issue Owner: @grgia

m-natarajan avatar Apr 02 '24 22:04 m-natarajan

Triggered auto assignment to @johncschuster (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

melvin-bot[bot] avatar Apr 02 '24 22:04 melvin-bot[bot]

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

MelvinBot avatar Apr 02 '24 22:04 MelvinBot

I'm not an approver on a workspace myself, so I won't be able to reproduce this. Applying the retest-weekly label and will ping the #QA room in Slack (per SO)

Edit: posted in Slack!

johncschuster avatar Apr 05 '24 18:04 johncschuster

Unable to reproduce

https://github.com/Expensify/App/assets/43996225/922761a4-b225-4f80-8189-a4f1c5795258

kavimuru avatar Apr 08 '24 01:04 kavimuru

@kavimuru can you try testing with Instant Submit and doing Scan Receipt instead of manual.

puneetlath avatar Apr 08 '24 14:04 puneetlath

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

melvin-bot[bot] avatar Apr 08 '24 18:04 melvin-bot[bot]

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

melvin-bot[bot] avatar Apr 09 '24 18:04 melvin-bot[bot]

I've bumped the Slack thread with your request, @puneetlath.

johncschuster avatar Apr 09 '24 20:04 johncschuster

@kavimuru indicated that they cannot reproduce this one.

johncschuster avatar Apr 12 '24 20:04 johncschuster

I can consistently reproduce this every day. How can I help troubleshoot?

puneetlath avatar Apr 15 '24 14:04 puneetlath

⚠️ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

melvin-bot[bot] avatar Apr 15 '24 18:04 melvin-bot[bot]

Job added to Upwork: https://www.upwork.com/jobs/~010d8f311682125da6

melvin-bot[bot] avatar Apr 15 '24 18:04 melvin-bot[bot]

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

melvin-bot[bot] avatar Apr 15 '24 18:04 melvin-bot[bot]

@johncschuster , snagging this from you cuz I'm working on documenting a process for issues that need reproduction. @eVoloshchak , can you see if you can reproduce plz? I create a job for $125 to provide reproduction steps, testing this out before I work on internally here.

mallenexpensify avatar Apr 15 '24 18:04 mallenexpensify

Sounds good, @mallenexpensify! Just a heads up, it looks like @puneetlath can consistently reproduce this every day, so I think we need to get clear on what the difference is between Puneet's reproduction setup and @kavimuru's setup.

johncschuster avatar Apr 15 '24 18:04 johncschuster

@eVoloshchak @mallenexpensify 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!

melvin-bot[bot] avatar Apr 16 '24 18:04 melvin-bot[bot]

Issue not reproducible during KI retests. (First week)

mvtglobally avatar Apr 18 '24 16:04 mvtglobally

Opened up to reproduction steps, leaving as-is for now.

mallenexpensify avatar Apr 18 '24 22:04 mallenexpensify

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

melvin-bot[bot] avatar Apr 22 '24 16:04 melvin-bot[bot]

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

melvin-bot[bot] avatar Apr 22 '24 18:04 melvin-bot[bot]

Bumped to weekly since we're looking for repro steps.

mallenexpensify avatar Apr 23 '24 21:04 mallenexpensify

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

melvin-bot[bot] avatar Apr 29 '24 16:04 melvin-bot[bot]

@eVoloshchak @mallenexpensify this issue is now 4 weeks old, please consider:

  • Finding a contributor to fix the bug
  • Closing the issue if BZ has been unable to add the issue to a VIP or Wave project
  • If you have any questions, don't hesitate to start a discussion in #expensify-open-source

Thanks!

melvin-bot[bot] avatar Apr 30 '24 18:04 melvin-bot[bot]

I still experience this every day. This is a core part of the Collect approver experience. What can I do to help diagnose? Would it be helpful to get device logs?

puneetlath avatar Apr 30 '24 22:04 puneetlath

@puneetlath is this the same or similar bug? I think my repro steps are

  1. Submit expense on collect policy
  2. As approver, observe the submitted expense doesn't show in LHN when in #focus mode

Submitted report image

In approver view, the request doesn't show when in #focus mode

https://github.com/Expensify/App/assets/22508304/2daa61db-a7fb-43ed-882d-c38736af2d0f

mallenexpensify avatar May 02 '24 17:05 mallenexpensify

Hmm, it could be similar, yes. What I consistently get is that the reports show up in the LHN, but they aren't bolded and they don't have the GBR.

Though I did just experience the same issue as you, where I got sent one but it's not showing up in my LHN at all.

puneetlath avatar May 02 '24 17:05 puneetlath

Upwork job price has been updated to $500

melvin-bot[bot] avatar May 02 '24 17:05 melvin-bot[bot]

Thanks @puneetlath , updated OP, title and price. Shared in #expensify-open-source too. Will try to prioritize this https://expensify.slack.com/archives/C01GTK53T8Q/p1714670946557159

mallenexpensify avatar May 02 '24 17:05 mallenexpensify

Here's an example from today. The chat in the bottom of the LHN from Nick is his workspace chat, which is there because he sent a request. But it is neither showing as unread or with the GBR. Screenshot 2024-05-03 at 2 37 11 PM

Once I click on it, the GBR arrives.

image

puneetlath avatar May 03 '24 18:05 puneetlath

Issue not reproducible during KI retests. (First week)

mvtglobally avatar May 06 '24 20:05 mvtglobally