App icon indicating copy to clipboard operation
App copied to clipboard

[Hold #51657][$250] Expense - Header shows skeleton and LHN shows Hidden when deleting expense with comment

Open lanitochka17 opened this issue 1 year ago • 19 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: 9.0.54-1 Reproducible in staging?: Y Reproducible in production?: Y If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5131224 Email or phone of affected tester (no customers): [email protected] Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to staging.new.expensify.com
  2. Go to FAB > Submit expense > Manual
  3. Submit expense to any user with no unsettled expense
  4. Go to transaction thread
  5. Send a message
  6. Click on the report header > Delete expense
  7. Delete the expense

Expected Result:

Header, LHN report title and thread parent message will show {Deleted message]

Actual Result:

  • Header shows skeleton
  • LHN report title shows Hidden
  • Thread parent message is empty

Workaround:

Unknown

Platforms:

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

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

Screenshots/Videos

Add any screenshot/video evidence

Uploading Bug6645540_1729860193572!20241025_203818.mp4…

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021851087295209337215
  • Upwork Job ID: 1851087295209337215
  • Last Price Increase: 2024-10-29
Issue OwnerCurrent Issue Owner: @trjExpensify

lanitochka17 avatar Oct 25 '24 14:10 lanitochka17

Triggered auto assignment to @trjExpensify (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.

melvin-bot[bot] avatar Oct 25 '24 14:10 melvin-bot[bot]

@trjExpensify 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 avatar Oct 25 '24 14:10 lanitochka17

We think that this bug might be related to #wave-collect - Release 1

lanitochka17 avatar Oct 25 '24 14:10 lanitochka17

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

melvin-bot[bot] avatar Oct 28 '24 18:10 melvin-bot[bot]

@lanitochka17 can I get a video of this bug please, as the one in the OP failed to attach.

trjExpensify avatar Oct 29 '24 02:10 trjExpensify

Alright, I can repro this:

https://github.com/user-attachments/assets/75ff2b5b-1c70-4019-b05a-2da3ae79a9ff

trjExpensify avatar Oct 29 '24 02:10 trjExpensify

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

melvin-bot[bot] avatar Oct 29 '24 02:10 melvin-bot[bot]

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

melvin-bot[bot] avatar Oct 29 '24 02:10 melvin-bot[bot]

I think a BE bug same root cause as https://github.com/Expensify/App/issues/51657#issuecomment-2444423334

FitseTLT avatar Oct 30 '24 10:10 FitseTLT

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

melvin-bot[bot] avatar Nov 01 '24 18:11 melvin-bot[bot]

Ah okay, so then I'll take Help wanted off and put it on hold for the other issue.

trjExpensify avatar Nov 04 '24 13:11 trjExpensify

Switched over the star on this issue to me for now, Melv.

trjExpensify avatar Nov 04 '24 13:11 trjExpensify

@trjExpensify, @allgandalf 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

melvin-bot[bot] avatar Nov 11 '24 15:11 melvin-bot[bot]

Still held.

trjExpensify avatar Nov 12 '24 01:11 trjExpensify

Still held..

allgandalf avatar Nov 14 '24 19:11 allgandalf

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

melvin-bot[bot] avatar Nov 15 '24 09:11 melvin-bot[bot]

Still held, dropping to weekly while we wait.

trjExpensify avatar Nov 15 '24 14:11 trjExpensify

No change.

trjExpensify avatar Nov 25 '24 17:11 trjExpensify

Dropping to Monthly until that gets picked up.

trjExpensify avatar Dec 04 '24 12:12 trjExpensify

@trjExpensify should i unassign myself here ?

allgandalf avatar Dec 05 '24 13:12 allgandalf

Can do man, probably going to be a hot minute.

trjExpensify avatar Dec 05 '24 19:12 trjExpensify

JLi is still after a volunteer on the issue this is held on, though potentially not reproducible anymore per here.

trjExpensify avatar Jan 03 '25 11:01 trjExpensify

Alright, the other issue isn't reproducible anymore. Asking for a retest.

trjExpensify avatar Feb 03 '25 22:02 trjExpensify

awaiting a retest.

trjExpensify avatar Feb 13 '25 19:02 trjExpensify

Issue not reproducible during KI retests. (First week)

mvtglobally avatar Feb 15 '25 00:02 mvtglobally

Cool, closing it out!

trjExpensify avatar Feb 18 '25 23:02 trjExpensify