-
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
[HOLD for payment 2024-06-11] CRITICAL: [Track Expense] [$250] Self DM doesn't scroll down when tracking new expense #40664
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01e9e386da4e15219d |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rojiphil ( |
Triggered auto assignment to @CortneyOfstad ( |
|
Upwork job price has been updated to $250 |
Contributor details This is my first proposal to work for Expensify. I've been able to test with the app, but not yet able to preview the video as I am waiting on my slack invite. |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
Contributor details |
|
ProposalPlease re-state the problem that we are trying to solve in this issue.Self DM doesn't scroll down when tracking new expense What is the root cause of that problem?Here, we're early returning (thus not scrolling user to the bottom) if But when we check This doesn't work some cases:
What changes do you think we should make in order to solve the problem?We should update this to use The only case where it does not What alternative solutions did you explore? (Optional)We can keep the strict equal condition, but will need to:
Currently the |
Contributor details |
📣 @sword1202! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
The issue at hand is that the self direct message (DM) chat window fails to scroll down when tracking a new expense. Root Cause: When the latest report action is of type To address this issue, we should modify the comparison logic to use a greater than or equal to (>=) check instead of strict equality. This change ensures that scrolling occurs if the created timestamp of the latest action is equal to or greater than Alternative Solutions Considered: Backend updates to ensure |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
@MelvinBot I haven't received yet any upwork job offer? So will wait If we mutually agree to a work contract. |
@saifelance you're new here. Please carefully read the contributing guidelines https://github.com/Expensify/App/blob/main/contributingGuides%2FCONTRIBUTING.md and every markdown file in the project to understand how contributors can work. |
@rojiphil, @marcochavezf, @CortneyOfstad, @tienifr Still overdue 6 days?! Let's take care of this! |
Bump @marcochavezf here. Thanks! |
@CortneyOfstad The comments as mentioned here are clarified already. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.78-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-06-11. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
@rojiphil can you have the checklist as soon as you have a chance? Just want to make sure there is no delays in issuing payments. Thank you! |
@CortneyOfstad This is a CRITICAL issue so according to the announcement, this should start at $500. Could you help to update the issue to reflect the same? TIA! |
Upwork job price has been updated to $500 |
@rojiphil any update on the checklist? Thanks! |
@CortneyOfstad There was clarification regarding the title here https://expensify.slack.com/archives/C01GTK53T8Q/p1717451725980889?thread_ts=1712256409.152489&cid=C01GTK53T8Q. |
I posted this last week in the thread you linked above @shubham1206agra, it's only for issues with
|
Upwork job price has been updated to $250 |
@rojiphil Any update on the checklist? |
@CortneyOfstad Accepted! |
|
@CortneyOfstad Completed BZ checklist and accepted offer. Thanks. |
QA Test Rail — https://github.com/Expensify/Expensify/issues/404214 |
Problem:
When viewing the self DM on desktop Chrome, and adding a new tracked expense, it should scroll down to show the new tracked expense just like when you post a message it will scroll down to show the new message. Similarly, when the actionable whisper comes after it, the screen should scroll down to show it. However, it doesn't.
Slack discussion (and video) here: https://expensify.slack.com/archives/C049HHMV9SM/p1713729858187919
Solution:
Two things:
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @CortneyOfstadThe text was updated successfully, but these errors were encountered: