Android - Conversation without messages is still displayed after navigating away

[isagoico]

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

---

Expected Result:

When navigating away, the conversation should not be displayed in the LHN if it doesn't have any messages.

Actual Result:

When navigating away from the conversation it is still displayed in the chat list

Action Performed:

1. Download the latest Android build open it and login
2. Tap on the + and select the new chat option
3. Search for a user that you don't have messages with
4. Open the chat
5. Without sending any message, navigate away from chat.

Workaround:

Unknown

Platform:

Where is this issue confirmed?

Web
iOS
Android ✔️
Desktop App
Mobile Web

Version Number: 1.0.7-0

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:
Unable to check if the issue is reproducible in the production app.
I Checked the behaviour in Web and the conversation does disappear when navigating away

Whatsapp.Video.2021-03-26.At.8.36.53.Pm-1.mp4

Expensify/Expensify Issue URL:

[MelvinBot]

Triggered auto assignment to @johncschuster (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[MelvinBot]

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

[joelbettner]

I think this has what it needs to be worked on by an engineer. Un-assigning myself for triage.

[mallenexpensify]

@joelbettner "I think this has what it needs to be worked on by an engineer. Un-assigning myself for triage."
When you say engineer do you mean an external contributor? I'm guessing yes but want to be sure

[mallenexpensify]

Double checking with Joel before posting

[joelbettner]

@mallenexpensify yes, external contributor or Expensify engineer. Either should be fine.

[mallenexpensify]

@johncschuster tossing back to you since you were the original auto-assigned triager, can you post to Upwork plz.

[mallenexpensify]

Strike that, I should add the External label to get someone from the CM team assigned, NOT a triager. Sorry @johncschuster

[MelvinBot]

Triggered auto assignment to @kevinksullivan (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[parasharrajat]

It seems like correct behavior. Actually, that chat is currently active(which means if you slide right you would see that report)so it is visible in LHN. if you switch to another chat then it would be removed from LHN.

@shawnborton But it does trigger one question, Do we need to highlight the active chat in some way on LHN in mobile devices?

[isagoico]

Issue reproducible during today's KI retests

[isagoico]

Issue reproducible during today's KI retests

[isagoico]

Issue reproducible during today's KI retests

[isagoico]

Issue reproducible today during KI retests

[isagoico]

Issue reproducible today during KI retests

[isagoico]

Issue reproducible today during KI retests

[kevinksullivan]

Sorry I missed this one. I also think it is ok for the conversation to remain if it disappears once no longer "active". That said I think consistency across platforms is important here. @isagoico does iOS immediately disappear?

[isagoico]

Issue reproducible today during KI retests.

[isagoico]

Yes, iOS has the same behaviour. So when I reproduced the issue I think this may be the expected behaviour. The conversation does disappear when you navigate to another conversation. On mobile it looks like you're navigating away but you need to navigate to another conversation to "close" the other one.

[kevinksullivan]

Got it. I think we'll close this one out for now then. Feel free to reopen if I am missing anything!