-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$250] Oldest unread message not displayed in Concierge #53169
Comments
Triggered auto assignment to @JmillsExpensify ( |
I'm almost certain this is by design. @garrettmknight do you bring this up in Slack? |
Bringing it up in the weekly update. |
@JmillsExpensify Eep! 4 days overdue now. Issues have feelings too... |
Sounds good. I'm still not sure that this is a real bug. |
@JmillsExpensify Eep! 4 days overdue now. Issues have feelings too... |
@garrettmknight do you mind clarifying what you're trying to get at in this issue? |
@JmillsExpensify 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! |
Current assignee @garrettmknight is eligible for the NewFeature assigner, not assigning anyone new. |
Job added to Upwork: https://www.upwork.com/jobs/~021866434424819479758 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @brunovjk ( |
In this specific case, we're showing the most recent message that is sent by concierge, skipping past important context in the oldest message when a new user signs up on mobile. More broadly, I think we should open to the oldest unread chat for a user to ensure that they're not missing context in the conversation. |
Also, this might be an internal issue, but I wanted to give external a shot before recruiting. |
Please correct me if Im wrong, but, I think this behavior might be intentional, as I worked on a similar issue (#52025) weeks ago. |
Gonna try to find where this is definitively written down before moving forwad. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.67-0
Reproducible in staging?: Y
Reproducible in production?: Y
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: @garrettmknight
Slack conversation (hyperlinked to channel name): ts_external_expensify_convert
Action Performed:
Link for session: https://fsty.io/v/ujphfF43
Expected Result:
Oldest unread message displayed in Concierge
Actual Result:
Oldest unread message not displayed in Concierge, scroll to the bottom of the setup tasks/message
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?
Screenshots/Videos
Add any screenshot/video evidence
task.con.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @garrettmknightThe text was updated successfully, but these errors were encountered: