-
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] Web - Assignee email or name is missing for other users #27144
Comments
Triggered auto assignment to @flaviadefaria ( |
Bug0 Triage Checklist (Main S/O)
|
Job added to Upwork: https://www.upwork.com/jobs/~018f80383d65e746df |
Triggered auto assignment to @sakluger ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ArekChr ( |
Stop emitting the 'assign event' when a user is assigned, and emit it only when a message is sent. Your Expensify account email: https://new.expensify.com/a/15606409 |
📣 @h0u554m! 📣
|
Stop emitting the 'assign event' when a user is assigned, and emit it only when a message is sent. Contributor details |
|
Stop emitting the 'assign event' when a user is assigned, and emit it only when a message is sent. Contributor details |
1 similar comment
Stop emitting the 'assign event' when a user is assigned, and emit it only when a message is sent. Contributor details |
We're waiting for proposals. |
Triggered auto assignment to @kadiealexander ( |
@kadiealexander I'm going OoO for a week so assigning this to you so that you can move it forward in the meantime. I'll take it back from you once I'm online again. Thanks! |
I also can no longer reproduce this error. @gadhiyamanan could you please confirm if you can still reproduce it? |
it is still reproduciable Screen.Recording.2023-09-20.at.10.56.35.AM.mov |
@kmostafa599 What is the root cause of the issue? Please follow the proposal template |
@ArekChr The current implementation of the setAssigneeValue function in codebase generates random IDs and uses a fake owner email and owner account when creating an optimistic chat report. This approach can lead to inconsistencies and potential issues in our application, as the generated IDs and owner details do not correspond to actual user data. The root cause of this problem is the use of randomly generated IDs and fake owner details when creating an optimistic chat report. This can lead to inconsistencies as these details do not match with actual user data. modify the setAssigneeValue function to use the proper user email instead of a fake one. This will involve the following steps: Retrieving User Data: When the setAssigneeValue function is called, we will retrieve the actual user data from our database or authentication service. This data should include the user’s email and account ID. Modifying Chat Report Generation: In the buildOptimisticChatReport function, will replace the randomly generated IDs and fake owner email with the actual user data retrieved in step 1. This will ensure that the chat report is linked to an actual user account. Updating Share Destination: If a share destination isn’t already set in this step, will set the shareDestination as that chat report. |
@ArekChr any further thoughts? |
@ArekChr @kadiealexander @flaviadefaria 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? 💸 |
Issue not reproducible during KI retests. (First week) |
@gadhiyamanan can you still reproduce this? I've never been able to. |
still able to reproduce Screen.Recording.2023-09-28.at.10.43.01.AM.mp4 |
@gadhiyamanan the video doesn't match the steps in the OP, could you please rewrite new reproduction steps? |
I am doing the same thing, which step am i missing ? |
I'm not sure, I just can't reproduce. When I try, I can't assign the task to a third user. All are gmail accounts. 2023-09-29_16-25-03.mp4 |
try this
Untitled.mp4 |
@ArekChr @kadiealexander @flaviadefaria this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
@ArekChr, @kadiealexander, @flaviadefaria Whoops! This issue is 2 days overdue. Let's get this updated quick! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
I tried reproducing this and also couldn't reproduce it. @kadiealexander @ArekChr have you been able to reproduce this with @gadhiyamanan last steps? |
Same here, couldn't reproduce this issue, when reassigning tasks to someone else I see the correct avatar and display name |
Ok since no one seems to be able to reproduce this I'll close this for now. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
Assignee email or name should show
Actual Result:
Assignee email or name is missing
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.67.0
Reproducible in staging?: y
Reproducible in production?: y
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
Notes/Photos/Videos: Any additional supporting documentation
Screen.Recording.2023-09-11.at.11.03.09.AM.mov
Recording.4378.mp4
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693807403315129
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: