Web - LHN - Conversation that doesn't have a message doesn't disappear from LHN

[kavimuru]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Go to https://staging.new.expensify.com/
2. Login with any account
3. Search for a user that you don't have any messages with and open the chat. Don't send any message
4. Navigate away from the conversation by clicking any other available conversation in the LHN list

Expected Result:

Conversation with no messages should disappear from LHN conversation list

Actual Result:

Conversation that doesn't have any messages doesn't disappear from from LHN

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• Desktop App

Version Number: v1.2.12-0
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Bug5765545_video_30.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:

View all open jobs on GitHub

[OSBotify]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[melvin-bot]

Triggered auto assignment to @johnmlee101 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[johnmlee101]

Can reproduce, looking why

[johnmlee101]

PR is up!

[kbecciv]

@johnmlee101 QA team is experienced the same issue on build 1.2.18.2

LHN.mp4

[sketchydroide]

It seems this is already happening in Prod removing blocker

[sketchydroide]

Is this a regression @johnmlee101? I set it as daily, as it's not a blocker

[johnmlee101]

@tgolen @ctkochan22 #11725
As per this change, we're okay leaving no comment standard chats in the LHN? I just realized this might have slipped through if this was unintended 😓

[tgolen]

Let's have @JmillsExpensify or @trjExpensify chime in on exactly what's expected. I don't think we had a clear idea before.

[trjExpensify]

Sorry for the delay, I've been OoO until today! (I'm also a poet.. 😄).

As per this change, we're okay leaving no comment standard chats in the LHN?

To clarify, we're not talking about archived chats like we were here? We're looking at..

1. CMD+F to a DM with someone you haven't messaged before
2. Navigate away from the chat without writing a message
3. The empty chat remains in the LHN

If that's correct, then I agree we shouldn't keep those empty DMs in the LHN.

[JmillsExpensify]

Ooh oops I missed this as well. That said, I agree with @trjExpensify that we couldn't keep empty DMs in the LHN>

[JmillsExpensify]

Huh, interesting. What's going on with this issue? Did we ever circle back and push an updated PR?

[flodnv]

+1 😄 we should probably wait until #13324 is done to revisit this.

However, I disagree with what is written in the issue body. This is not a bug, the actual behavior is the expected behavior, IMO. I vote we close this.

[flodnv]

I agree with @trjExpensify that we couldn't keep empty DMs in the LHN

I disagree with that, I don't really see a problem with this. What's the actual problem?

[trjExpensify]

My rationale personally is that your LHN is sorted by "most recent chats". There's nothing recent about an empty chat nobody has chatted in, so why does it continue to show with no activity?

[flodnv]

Maybe.. I feel like this is really a detail that does not matter much. In your thinking, when you create a new chat with someone you've never chatted to before and do nothing, you have the chat open waiting for you to write something -- does it show in the LHN or not? I think either answer to that question is both wrong and right 😄 If we want to mimic Whatsapp then it won't appear in the LHN until there's a message in it, which I think is just as fine as the other option.

[flodnv]

Maybe let's open a new, clean issue?

[trjExpensify]

do not show a chat for which the only reportAction is the CREATED one.

Question on that though for my understanding.. if you're typing a draft message is there still only a CREATED reportAction though?

[flodnv]

Hm yes, but we do agree that if there is a draft message, it should show show in the LHN.

[trjExpensify]

Right, we do.. so "do not show a chat for which the only reportAction is the CREATED one." isn't wholly accurate for our intentions with this then? 😅

[flodnv]

Correct. I think we want "Do not show a chat in the LHN for which the only reportAction is the CREATED one and does not have a draft."

Said another (non french) way, in the LHN we should show chats that have at least 1 ADDCOMMENT action, OR 1 draft.

[trjExpensify]

Cool, so I think a new issue with the following is where we landed. Will you be taking it, John, or someone else?

Title: Show chats in the LHN that have at least 1 ADDComment action, or 1 draft message.

Action Performed:

1. Go to https://staging.new.expensify.com/
2. Login with any account
3. Search for a user that you don't have any messages with, open the chat but don't send any message

Expected results:
Chat should not show in the LHN as there is not at least 1 ADDComment action, or 1 draft message.

Actual results:
Empty chat shows in the LHN and remains in the LHN after simply navigating to it.

[flodnv]

👍

[JmillsExpensify]

Missed the more recent discussion, though I agree with where we landed. Did we create this issue yet?

[trjExpensify]

I didn't get to this, so if someone wants to create it, go for it. Else, I can later next week when I'm back.

…

On Wed, Jan 18, 2023 at 4:42 PM Jason Mills ***@***.***> wrote: Missed the more recent discussion, though I agree with where we landed. Did we create this issue yet? — Reply to this email directly, view it on GitHub <#11633 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AD3246K4WYTA7KIDB6X5BILWTAMQHANCNFSM6AAAAAAQ62D7TI> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- *Tom Rhys Jones * *Expensify*

[JmillsExpensify]

Cool, I can get to it tomorrow. Leaving a tab open for this.

[johnmlee101]

Not overdue!

[JmillsExpensify]

Related issue created here: #14523

[JmillsExpensify]

Is there anything else left to do here? I kind of lost where we're at for this issue.

[flodnv]

Thanks @JmillsExpensify , we can close this now

[lanitochka17]

Hello
Issue reproducible. Windows10/Chrome. Build 1.3.37.1
Thanks

Recording.322.mp4

[melvin-bot]

@johnmlee101 Eep! 4 days overdue now. Issues have feelings too...

[johnmlee101]

I think this is a separate issue. Do we want welcome messages to show as unread on the LHN? Can you create a new issue to address this?

[melvin-bot]

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

[johnmlee101]

Closing to create a new issue

[flodnv]

Maybe we are solving this here #19321 ?

[johnmlee101]

Yeah it seems relevant to the original issue #14523