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

Deleted Tasks inside threads are counted in the thread reply count #39016

Closed
1 of 6 tasks
m-natarajan opened this issue Mar 26, 2024 · 5 comments
Closed
1 of 6 tasks

Deleted Tasks inside threads are counted in the thread reply count #39016

m-natarajan opened this issue Mar 26, 2024 · 5 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@m-natarajan
Copy link

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: 1.4.56-2
Reproducible in staging?: y
Reproducible in production?: y
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: @fedirjh
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1711463195704049

Action Performed:

  1. Open a chat and Start a new thread
  2. Create a new Task inside the thread
  3. Navigate to task and delete it
  4. Go back to main chat

Expected Result:

The correct number of replies is displayed for every thread

Actual Result:

Deleted task is counted in the thread reply count

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
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Recording.2906.mp4
CleanShot.2024-03-25.at.23.13.16.mp4

View all open jobs on GitHub

@m-natarajan m-natarajan added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 26, 2024
Copy link

melvin-bot bot commented Mar 26, 2024

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

@allgandalf
Copy link
Contributor

I see deleted task message on my end so maybe it is intentional :

image

But if we don't want that then we need to call to update childVisibleActionCount:

function updateOptimisticParentReportAction(parentReportAction: OnyxEntry<ReportAction>, lastVisibleActionCreated: string, type: string): UpdateOptimisticParentReportAction {

@greg-schroeder
Copy link
Contributor

I think this is expected and makes sense to me - the task is deleted but its existence is maintained by the deletion text, so I'm going to close this for now to focus on higher prio roadmap items

@fedirjh
Copy link
Contributor

fedirjh commented Mar 27, 2024

the task is deleted but its existence is maintained by the deletion text,

@greg-schroeder @GandalfGwaihir This is not the case for me. The deleted text tag disappears after some time.

CleanShot.2024-03-27.at.01.08.30.mp4

@bernhardoj
Copy link
Contributor

@fedirjh oh, I just found out that we already have an issue for this here, so we can keep this closed.

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
Projects
None yet
Development

No branches or pull requests

5 participants