[$1000] Visiting room by pasting URL, doesn't read messages - reported by @K4tsuki

[mvtglobally]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Action Performed:

1. Close all Expensify browser tabs.
2. Send a message to a room using other device
3. Visit the room directly by url (for example new.expensify.com/r/XXXXXX ) or by home url if the room is last visited.
4. Verify that room style keep displayed as unread and unread line indicator (green line) keep being shown even after user scroll up and down in the message. Even after user navigate to other room, room style keep displayed as unread.

Expected Result:

Room style must be displayed as read (not in bold) and unread indicator line will disappear after user scroll chat up and down.

Actual Result:

Room style keep displayed as unread and unread line indicator (green line) keep being shown even after user scroll up and down in the message. Even after user navigate to other room, room style keep displayed as unread.

Workaround:

unknown

Platform:

Where is this issue occurring?

• Web
• Mobile Web

Version Number: 1.1.39-1
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

marker.mp4

Expensify/Expensify Issue URL:
Issue reported by: @K4tsuki
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1644253704477339

View all open jobs on GitHub

[MelvinBot]

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

[flaviadefaria]

Makes sense to fix this. I also tested in Slack and after a few seconds in the room, the conversation is "marked as read" and the room name is no longer flagged in bold.

[MelvinBot]

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

[joelbettner]

Yeah, this should definitely be fixed. Un-assigning myself and labeling as external.

[MelvinBot]

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

[mateusbra]

maybe duplicate of #6471

[MelvinBot]

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

[K4tsuki]

Is this duplicate issue? Could I submit a proposal?

[MelvinBot]

6 days overdue. This is scarier than being forced to listen to Vogon poetry!

[MelvinBot]

Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it!

[MelvinBot]

10 days overdue. Is anyone even seeing these? Hello?

[MelvinBot]

This issue has not been updated in over 14 days. eroding to Weekly issue.

[mallenexpensify]

Sorry for the spammin'
The CM originally got removed by mistake so I'm hopping in (then... somehow removed folks, I'm blaming GH)
@parasharrajat , can you check #6471 to see if it's related to this?

[parasharrajat]

It does not look related. But we can ask QA to retest it when #6471 is on staging.

[mallenexpensify]

Doubled price to $500
https://www.upwork.com/jobs/~0108a6d5655ee24bf8

[pecanoro]

Not overdue!

[mallenexpensify]

Doubled price to $1000
https://www.upwork.com/jobs/~0108a6d5655ee24bf8

[mateusbra]

After #6471 was deployed to staging I was unable to reproduce it from my side, I think we could tag some QA and see if it's still reproducible.

It does not look related. But we can ask QA to retest it when #6471 is on staging.

I think even its not being related we ended up also solving this one in #6471

Just need to confirm with some QA

[mallenexpensify]

@isabelastisser @mvtglobally can you test to see if you're able to reproduce? Thanks!

[isabelastisser]

Hey @mallenexpensify , did you mean to tag @isagoico? :)

[mallenexpensify]

Indeed! Sorry @isabelastisser (I'm surprised this doesn't happen 10x as much)

[mallenexpensify]

@isagoico and @mvtglobally , are you able to reproduce?

[mateusbra]

@isagoico I think that for this issue you should open it directly by pasting the URL.
The steps would be:

1. Open the report as user A.
2. Copy the report URL and close the tab on browser.
3. Send messages from user B to user A.
4. Open the report directly by pasting the copied URL.

Is that right?

cc: @parasharrajat
cc: @mvtglobally

[isagoico]

I was unable to reproduce the same issue on my side following those steps. Please check the video and let me know if the steps are incorrect.

Recording.4.mp4

[mateusbra]

You're right, that's the correct steps. But I think that's the expected behavior, isn't?

Expected Result:

Room style must be displayed as read (not in bold) and unread indicator line will disappear after user scroll chat up and down.

[isagoico]

Yeap, the issue is not reproducible. Behavior was the expected on my side.
I was able to reproduce a similar issue using the URL but I don't have solid reproducible steps, will keep investigating and trying to replicate again🕵️

[pecanoro]

If no one can reproduce it, let's close it!