-
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
[$1000] Task- Assignee selected is not updated in real time in user A task detail page for sub task #30015
Comments
Triggered auto assignment to @kevinksullivan ( |
Job added to Upwork: https://www.upwork.com/jobs/~015c5e0dff9ee32231 |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane ( |
@kevinksullivan, @rushatgabhane Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@kevinksullivan, @rushatgabhane Huh... This is 4 days overdue. Who can take care of this? |
📣 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 $1000 |
Doubling price |
@kevinksullivan This issue seems to be a BE issue rather than a front-end issue. See attached screen recording (iOS is User A, Web is User B). When adding a new sub-task, it does show up on User A immediately, as expected. But once you assign the task to yourself, note that the log screen is empty, meaning we received nothing from the pusher. No pusher data means no UI updates on User A. Once we finally assign the task to account A, we see pusher activity in the logs and the UI reflects the new assignment. It appears that the BE will look at the sub-task and think it has nothing todo with User A. In fact, looking at it in isolation, it doesn't really. The link between the sub-task and User A is through the parent task. Maybe the BE is not taking that into account? It looks at the sub-task in isolation and ignore the parent relationship and therefore never pushes data to User A? There's also another problem, at the end of the video. After assigning the sub-task to User A, if you later assign it to a 3rd user, User A UI will update, but it will clear the assignment instead of showing the 3rd user. Either way, doesn't look like this can be fixed on the FE. screen-recording.mov |
@kevinksullivan, @rushatgabhane Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Triggered auto assignment to @madmax330 ( |
Looping in engineering to confirm ^ |
@madmax330 @kevinksullivan @rushatgabhane this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
cc @thienlnam penny for your thoughts? Since you worked on the tasks implementation |
Ah yeah, this looks like we're not sending a pusher update to update the reportAction of the task parent on the assignee update - I can spin up a PR for this EDIT: Wait, does this also happen for the task name? It seems like it would from looking at the code if User B has not opened the task yet |
@thienlnam made the fix for this. |
@kevinksullivan, @rushatgabhane, @thienlnam Still overdue 6 days?! Let's take care of this! |
Let's verify if this is still happening from the BE fix and if not we can close |
@kevinksullivan, @rushatgabhane, @thienlnam Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@kevinksullivan, @rushatgabhane, @thienlnam Huh... This is 4 days overdue. Who can take care of this? |
Any update @thienlnam ? |
@lanitochka17 Could we please get this one retested? |
@kevinksullivan, @rushatgabhane, @thienlnam Eep! 4 days overdue now. Issues have feelings too... |
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: 1.3.87-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
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: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
The assignee selected must be updated in real time in user A task detail page
Actual Result:
The assignee selected is not updated in real time in user A task detail page. When user taps on task and tap header and revisit the page only assignee is updated and shown
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Android: Native
Bug6243418_1697740718079.Android_user.mp4
Uploading Bug6243418_1697740947925!android_user.mp4…
Android: mWeb Chrome
Bug6243418_1697740718062.video1_redmi.mp4
Bug6243418_1697740791106.video2.mp4
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: