Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.1k stars 2.6k forks source link

[$250] IOU - Members page is blank #44040

Open lanitochka17 opened 1 week ago

lanitochka17 commented 1 week ago

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: 1.4.85-6 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

Action Performed:

  1. Log into staging.new.expensify.com
  2. Navigate to a conversation
  3. Create an IOU
  4. Click on the header of the IOU to access the members

Expected Result:

A members page/ modal should open listing all the members

Actual Result:

The modal is empty (blank)

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/cf7c5f36-dae9-46fb-a2d1-7cd0fb6db1f9

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~017fcedc031d85e839
  • Upwork Job ID: 1805044099624774803
  • Last Price Increase: 2024-07-01
Issue OwnerCurrent Issue Owner: @allgandalf
melvin-bot[bot] commented 1 week ago

Triggered auto assignment to @VictoriaExpensify (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 commented 1 week ago

@VictoriaExpensify 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

bernhardoj commented 1 week ago

Proposal

Please re-state the problem that we are trying to solve in this issue.

Member list page is empty/blank when pressing the multiple display name of a report preview action.

What is the root cause of that problem?

In this old PR, we navigate the user to the report participants page if it's an IOU report (which it is in our case). We do this for both AvatarWithDisplayName (used in the IOU report header) and ReportActionItemSingle display name.

However, all participants are hidden, so when we get the member list, we get an empty list. https://github.com/Expensify/App/blob/75614394a2fb1e8114e35bb2d8d33bdd0c565946/src/pages/ReportParticipantsPage.tsx#L72-L73 https://github.com/Expensify/App/blob/75614394a2fb1e8114e35bb2d8d33bdd0c565946/src/libs/ReportUtils.ts#L1850-L1856

If we see the AvatarWithDisplayName case, we now update it so that for the IOU report, we navigate to the report detail instead of the participants page. https://github.com/Expensify/App/blob/75614394a2fb1e8114e35bb2d8d33bdd0c565946/src/components/MoneyReportHeader.tsx#L209-L211 https://github.com/Expensify/App/blob/75614394a2fb1e8114e35bb2d8d33bdd0c565946/src/components/AvatarWithDisplayName.tsx#L78-L82

Also, the IOU report detail page doesn't have a members section.

image

What changes do you think we should make in order to solve the problem?

I suggest changing the navigation to the report details page too for the IOU report. https://github.com/Expensify/App/blob/75614394a2fb1e8114e35bb2d8d33bdd0c565946/src/pages/home/report/ReportActionItemSingle.tsx#L163-L167

Navigation.navigate(ROUTES.REPORT_WITH_ID_DETAILS.getRoute(iouReportID));
VictoriaExpensify commented 1 week ago

I agree this looks buggy and have been able to reproduce the issue:

image

I believe this belongs to Project VSB

melvin-bot[bot] commented 1 week ago

Job added to Upwork: https://www.upwork.com/jobs/~017fcedc031d85e839

melvin-bot[bot] commented 1 week ago

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

allgandalf commented 5 days ago

@bernhardoj , thanks for your proposal, your RCA is correct, but the expected results from this issue is to show the members page:

A members page/ modal should open listing all the members

@VictoriaExpensify , can you confirm the expected result here? I agree with @bernhardoj that we should show the reports detail page here instead of members page for IOU, can you confirm this here please :) thanks

allgandalf commented 1 day ago

friendly bump @VictoriaExpensify

melvin-bot[bot] commented 1 day ago

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