-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$500] Chat - Chat is not scrolled to the new message on a newly created WS chat after whisper message #38855
Comments
👋 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:
|
Triggered auto assignment to @robertjchen ( |
@robertjchen FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
I think same RCA as #38848. |
Job added to Upwork: https://www.upwork.com/jobs/~0195afebb1a07596ef |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia ( |
Yep, this can be handled as an External task 👍 |
Upwork job price has been updated to $250 |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Upwork job price has been updated to $500 |
updated |
Awaiting proposals |
ProposalsCross posting my proposal from other issue, I think this issue has same RCA. |
@robertjchen, @anmurali Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Looks like it may be resolved by #40724 (comment) 👍 |
@robertjchen, @anmurali Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Pending solution in #40724 (comment) |
Issue not reproducible during KI retests. (First week) |
This issue has not been updated in over 15 days. @robertjchen, @anmurali 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! |
Nice, looks like the change was deployed there. Given that it's unreproducible here, I'll close this out in favor of the one there 👍 #40724 (comment) |
I might be eligible for some compensation on this issue. Reason:
Reviewed PR: #40286 |
@luacmartins and @mountiny. @parasharrajat has requested payment of 50% for this issue because they worked on the PR.
I'm going to pay later this week, assuming neither of you disagree. |
Payment requested as per #38855 (comment) |
$250 approved for @parasharrajat |
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: 1.4.56-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
The chat should scroll down to the new sent message
Actual Result:
Chat page does not scroll to the bottom to the new message sent after the whisper message
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6423703_1711139703006.Screen_Recording_20240322_232619_Chrome.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: