You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Room - Offline created custom room does not display default avatar #72326
👋 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:
Identify the pull request that introduced this issue and revert it.
Please re-state the problem that we are trying to solve in this issue.
Room - Offline created custom room does not display default avatar
What is the root cause of that problem?
It's causing the ?? (nullish coalescing) operator only replaces the value when the left-hand side is null or undefined — not when it’s an empty string (""), 0, or false.
In this case,report.avatarUrl ends up taking an empty string. Because the avatar is never uploaded.
What alternative solutions did you explore? (Optional)
N/A
Result
Screen.Recording.2025-10-10.at.21.31.49.mp4
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.
Activity
melvin-bot commentedon Oct 10, 2025
Triggered auto assignment to @brianlee-expensify (
Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.melvin-bot commentedon Oct 10, 2025
Triggered auto assignment to @flodnv (
DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.melvin-bot commentedon Oct 10, 2025
💬 A slack conversation has been started in #expensify-open-source
github-actions commentedon Oct 10, 2025
👋 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:
andriferry commentedon Oct 10, 2025
Proposal
Please re-state the problem that we are trying to solve in this issue.
Room - Offline created custom room does not display default avatar
What is the root cause of that problem?
It's causing the
??(nullish coalescing) operator only replaces the value when the left-hand side isnullorundefined— not when it’s an empty string(""), 0, or false.In this case,
report.avatarUrlends up taking an empty string. Because the avatar is never uploaded.App/src/components/RoomHeaderAvatars.tsx
Line 54 in 941846d
What changes do you think we should make in order to solve the problem?
Just change to conditional (ternary) operator like this
What alternative solutions did you explore? (Optional)
N/A
Result
Screen.Recording.2025-10-10.at.21.31.49.mp4
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.
brianlee-expensify commentedon Oct 10, 2025
confirmed that i can reproduce on staging.new.expensify.com
brianlee-expensify commentedon Oct 10, 2025
@flodnv hello! any thoughts on next steps here?
10 remaining items
mountiny commentedon Oct 13, 2025
@samranahm @Beamanator @eh2077 comes from #66883 most likely
Beamanator commentedon Oct 15, 2025
PR was reverted, looks like next PR is WIP
eh2077 commentedon Oct 21, 2025
discussion is ongoing in the parent issue #65210
eh2077 commentedon Oct 28, 2025
commented in the parent issue #65210 (comment)
eh2077 commentedon Nov 14, 2025
@Beamanator I think this can be closed.