Concierge chat shows only most recent message and all other turns into an infinite skeleton
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: Reproducible in staging?: Need reproduction Reproducible in production?: need reproduction If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: 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: @trjExpensify Slack conversation (hyperlinked to channel name): expensify-quality
Action Performed:
- Open ND
- Go to Concierge chat
Expected Result:
All chats should be loading
Actual Result:
Concierge chat initially loads, then all but the most recent message turns into an infinite skeleton
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
- [ ] Android: Standalone
- [ ] Android: HybridApp
- [ ] Android: mWeb Chrome
- [ ] iOS: Standalone
- [ ] iOS: HybridApp
- [ ] iOS: mWeb Safari
- [x] MacOS: Chrome / Safari
- [ ] MacOS: Desktop
Screenshots/Videos
Add any screenshot/video evidence
Issue Owner
Current Issue Owner: @janicduplessis
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989
Triggered auto assignment to @RachCHopkins (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.
Triggered auto assignment to @techievivek (AutoAssignerNewDotQuality)
I can't repro - Replied in thread.
@RachCHopkins, @techievivek Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
Per the above thread @janicduplessis is going to take a look at this.
@janicduplessis Can you please comment on the GH so I can assign it to you, thanks.
I will have a look at this today
I'm having trouble reproducing this one.
Posted in slack:
Is anyone able to reproduce this on a new account? I've had no luck there. From looking at the code the only way this kind of skeleton can show is when the state isLoadingInitialReportActions or isLoadingOlderReportActions is true, but I don't see any calls to GetOlderReportActions in the logs so it must be the first case. This is set to true when calling OpenReport I've looked at the OpenReport calls from the logs for report id 4084426685399241 which is one of the problematic ones in the posted screenshots, it is hard to see exactly what is going on, I wonder if some race condition in SequentialQueue could cause the OpenReport response to never be processed.
I wasn't able to repro it, but I've seen it multiple time in FS reviews.
@janicduplessis, @RachCHopkins, @techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick!
@janicduplessis @RachCHopkins @techievivek 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!
@janicduplessis, @RachCHopkins, @techievivek Huh... This is 4 days overdue. Who can take care of this?
@janicduplessis I know you said in Slack that you were going to get into this again, how are you getting on?
Investigating this, @muttmuure said he has more cases to show me. Currently we think it might be related to OpenReport call never resolving.
@janicduplessis, @RachCHopkins, @techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick!
Not overdue. @janicduplessis, are you looking into this? Any help you need from the backend for this?
@janicduplessis, @RachCHopkins, @techievivek Eep! 4 days overdue now. Issues have feelings too...
Calm down Melvin, it's the holidays... 🎄
@janicduplessis, @RachCHopkins, @techievivek Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
Fix linked in thread, going to close!
@janicduplessis, @RachCHopkins, @techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick!
@muttmuure why was this reopened ?
@janicduplessis, @RachCHopkins, @techievivek Still overdue 6 days?! Let's take care of this!
@janicduplessis, @RachCHopkins, @techievivek 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!
@janicduplessis, @RachCHopkins, @techievivek 12 days overdue now... This issue's end is nigh!
This issue has not been updated in over 14 days. @janicduplessis, @RachCHopkins, @techievivek eroding to Weekly issue.
This issue has not been updated in over 15 days. @janicduplessis, @RachCHopkins, @techievivek eroding to Monthly issue.
P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!