search

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.4k stars 2.79k forks source link

[HELP WANTED] Task - Reply count not updated when task is assigned and deleted in thread #46729

Open lanitochka17 opened 2 months ago

lanitochka17 commented 2 months 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: 9.0.16-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 Email or phone of affected tester (no customers): abebemiherat@gmail.com Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to any chat and write a message
  2. In the thread, click the plus sign and assign a task
  3. Add a title, assignee, and confirm
  4. Exit the thread and observe the updated reply count
  5. Re-enter the thread, click on the task header, and delete the task
  6. Exit the thread and notice the reply count is not updated

Expected Result:

The reply count should update when a task is deleted in the thread

Actual Result:

Reply count is not updated when a task is deleted in the thread

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/fb16b326-15e1-464e-9f24-8dcd8375c661

View all open jobs on GitHub

melvin-bot[bot] commented 2 months ago

Triggered auto assignment to @strepanier03 (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 2 months ago

@strepanier03 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 commented 2 months ago

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

FitseTLT commented 2 months ago

BE

melvin-bot[bot] commented 1 month ago

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

strepanier03 commented 1 month ago

I'm able to repro this following the instructions. Even if you delete the comment you threaded from, the reply count doesn't update so I'll let this go forward and see if eng wants to pick it up.

melvin-bot[bot] commented 1 month ago

@strepanier03 Huh... This is 4 days overdue. Who can take care of this?

strepanier03 commented 1 month ago

Setting to weekly until this get prioritized.

melvin-bot[bot] commented 1 month ago

@strepanier03 this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

strepanier03 commented 4 weeks ago

Hasn't been picked up yet.

strepanier03 commented 2 weeks ago

Waiting for prioritization and ownership.