Closed lanitochka17 closed 1 week ago
Triggered auto assignment to @joekaufmanexpensify (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.
@joekaufmanexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors
We think that this bug might be related to #vip-vsp
App notification shows only "New Expensify" instead of sender name when sender does not have a display name.
This is indeed a BE issue that when sender does not have a display name (or display name is the default which is equal to login), Pusher sends the action's person
with empty text
.
Later the browser engine would fallback the the app name if notification's title is empty.
However, we could handle that in the FE by using the action's actorAccountID
and find the corresponding display name using ReportUtils.getDisplayNameForParticipant
:
const {person, message, actorAccountID} = reportAction;
let plainTextPerson = person?.map((f) => f.text).join() ?? '';
if (!plainTextPerson && actorAccountID) {
plainTextPerson = ReportUtils.getDisplayNameForParticipant(actorAccountID);
}
Going to review this morning
This is reproducible on any platform. We're not showing the primary login in the notification if no display name is set.
Made this internal based on the above comment that the fix here is BE
Job added to Upwork: https://www.upwork.com/jobs/~0105129a47d0b5f423
Triggered auto assignment to Contributor Plus for review of internal employee PR - @dukenv0307 (Internal
)
@dukenv0307 LMK if you agree that BE fix is needed here?
https://github.com/Expensify/App/assets/129500732/ed0129dc-3535-47c2-b1b6-3fde5bbdc426
@joekaufmanexpensify I can't reproduce
Interesting. I'll try and reproduce again today
Hmm yeah it's working for me now too. Perhaps it was an issue related to our site instability yesterday. Going to close for now since this seems to be resolved!
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.70-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: https://expensify.testrail.io/index.php?/tests/view/4532040 Issue reported by: Applause - Internal Team
Action Performed:
Precondition:
Expected Result:
App notification will show User A's email as display name
Actual Result:
App notification shows only "New Expensify" instead of User A's email
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/78819774/808e7ec9-6530-4145-8876-e4fd2172df16
View all open jobs on GitHub
Upwork Automation - Do Not Edit