[Internal] Room - Room chat avatar is different for user who creates room & who has been invited to room

@grgia

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.15-4
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

Launch app
Tap fab--start chat
Tap room
Enter room name and select a workspace with custom avatar
Tap create room
Tap header
Tap members---Invite
Invite user X to room
9 .Navigate to LHN and note avatar of room chat
Go to https://staging.new.expensify.com/
Login as user X
Note avatar of room chat

Expected Result:

Room chat avatar must be same for user who creates room and for user who has been invited to room

Actual Result:

Room chat avatar is different for user who creates room and for user who has been invited to room

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

Screenshots/Videos

Add any screenshot/video evidence

Bug6323221_1703194294854.full.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013499a52b4634b0ed
Upwork Job ID: 1737991692071854080
Last Price Increase: 2023-12-29

Issue Owner

Current Issue Owner: @grgia

Job added to Upwork: https://www.upwork.com/jobs/~013499a52b4634b0ed

@alexpensify

Triggered auto assignment to @alexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
This bug is not a duplicate report (check E/App issues and #expensify-bugs)
- If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
This bug is reproducible using the reproduction steps in the OP. S/O
- If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
- If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
This issue is filled out as thoroughly and clearly as possible
- Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@getusha

Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha (External)

Proposal

from: @unicorndev-727

Please re-state the problem that we are trying to solve in this issue.

Room chat avatar is different for user who creates room & who has been invited to room

What is the root cause of that problem?

The root cause is that the user who has been invited to room doesn't have the policy data about workspace in Onyx so that the default workspace avatar will be showed.
This issue only happens when we invite the user who isn't member of this workspace to the only room.

App/src/libs/ReportUtils.ts

Lines 1247 to 1249 in 8970aa3

    
           const policyExpenseChatAvatarSource = allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`]?.avatar 
        
               ? allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`]?.avatar 
        
               : getDefaultWorkspaceAvatar(workspaceName);

It is a screen shot of the room owner.

It is a screen shot of the room invited user.

What changes do you think we should make in order to solve the problem?

We can send the policy update request if ONYXKEYS.COLLECTION.POLICY}${report?.policyID} is null here.

What alternative solutions did you explore? (Optional)

Backend need to send the workspace data to the users who has been invited to the room and isn't the member of the workspace.

Proposal

Please re-state the problem that we are trying to solve in this issue.

Room chat avatar is different for user who creates room and for user who has been invited to room

What is the root cause of that problem?

We're using the workspace avatar as the policy room avatar.

But when the user is invited to a policy room, but the user is not a member of that workspace, they don't have access to that information. That's expected because if user is not in the workspace, they shouldn't have access to the policy record.

What changes do you think we should make in order to solve the problem?

We had the same issue before with the workspace name, and fixed it by adding the policyName as a field in the report of the room.

We should do the same here for the workspace avatar, we need to add a policyAvatar field in the report record, which will be synced with the avatar of the workspace, the back-end will also need to be updated to support this.

Then in front-end we'll use that field as a fallback in case the policy avatar is not found. If even that policyAvatar is not available, now we fallback to the default workspace avatar as we currently do.

This way, even though the user doesn't have any access to the workspace data, they can still access the policyAvatar (beside policyName) to display as room avatar.

What alternative solutions did you explore? (Optional)

NA

@getusha

@getusha - - when you get a chance, can you review if this proposal will fix this issue? Thanks!

Heads up, I will be offline from Friday, December 22, to Thursday, January 4, 2024. I will not be actively watching over this GitHub during that period. If anything urgent is needed here, please ask for help in the #expensify-open-source Slack Room-- thanks!

@unicorndev-727 do you mind adding more details to the solution you mentioned in your proposal?

@techievivek

@techievivek - we need your follow-up here. Thanks!

Issue not reproducible during KI retests. (Second week)

@alexpensify

I checked again and the bug is fixed on mweb on the latest main. Can the QA team check again? Correct me if I missed something.

QA team just confirmed that this is fixed so we can move forward with the payment here. @alexpensify

@dukenv0307

Payouts due: 2024-09-09

Contributor: $500 @dukenv0307
Contributor+: $500 @getusha

Upwork job is here.

@dukenv0307

@dukenv0307 - I sent an offer in Upwork. Please accept and I can complete the payment process.

Thanks!

@alexpensify

Please accept

@alexpensify Done, thank you

@alexpensify

@alexpensify i should get paid on Upwork as well since the issue was created before my ND payment eligibility date.

@getusha

@getusha - thanks for flagging, I sent over an offer in Upwork.

@dukenv0307

@dukenv0307 - thanks, I completed the payment process in Upwork!

@alexpensify

@alexpensify accepted the offer. ty!

Awesome, everyone has been paid via Upwork.

#33470 (comment)

Navigation Menu

Repository navigation

[Internal] Room - Room chat avatar is different for user who creates room & who has been invited to room #33470

Action Performed:

Expected Result:

Actual Result:

Workaround:

Platforms:

Screenshots/Videos

Bug0 Triage Checklist (Main S/O)

Proposal

Please re-state the problem that we are trying to solve in this issue.

What is the root cause of that problem?

What changes do you think we should make in order to solve the problem?

What alternative solutions did you explore? (Optional)

Proposal

Please re-state the problem that we are trying to solve in this issue.

What is the root cause of that problem?

What changes do you think we should make in order to solve the problem?

What alternative solutions did you explore? (Optional)

136 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	const policyExpenseChatAvatarSource = allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`]?.avatar
	? allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`]?.avatar
	: getDefaultWorkspaceAvatar(workspaceName);

[Internal] Room - Room chat avatar is different for user who creates room & who has been invited to room #33470

Description

Action Performed:

Expected Result:

Actual Result:

Workaround:

Platforms:

Screenshots/Videos

Activity

melvin-bot commented on Dec 22, 2023

melvin-bot commented on Dec 22, 2023

melvin-bot commented on Dec 22, 2023

Bug0 Triage Checklist (Main S/O)

melvin-bot commented on Dec 22, 2023

unidev727 commented on Dec 22, 2023

Proposal

Please re-state the problem that we are trying to solve in this issue.

What is the root cause of that problem?

What changes do you think we should make in order to solve the problem?

What alternative solutions did you explore? (Optional)

dukenv0307 commented on Dec 22, 2023

Proposal

Please re-state the problem that we are trying to solve in this issue.

What is the root cause of that problem?

What changes do you think we should make in order to solve the problem?

What alternative solutions did you explore? (Optional)

alexpensify commented on Dec 23, 2023

getusha commented on Dec 25, 2023

136 remaining items

alexpensify commented on Sep 6, 2024

mvtglobally commented on Sep 9, 2024

techievivek commented on Sep 9, 2024

alexpensify commented on Sep 9, 2024

alexpensify commented on Sep 9, 2024

dukenv0307 commented on Sep 10, 2024

getusha commented on Sep 10, 2024

alexpensify commented on Sep 10, 2024

alexpensify commented on Sep 10, 2024

getusha commented on Sep 11, 2024

alexpensify commented on Sep 11, 2024

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions