Trigger whisper when user invites a member to the chat #71280
Trigger whisper when user invites a member to the chat #71280carlosmiceli merged 21 commits intoExpensify:mainfrom
Conversation
Codecov Report❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.
|
|
@parasharrajat do you think we'll get back to this sometime this week? |
|
Yes. I did the changes. Just need to do some testing which will be done today. |
…ert-69844-parasharrajat/member
|
Created a discussion for handling navigation complexities https://expensify.slack.com/archives/C08CZDJFJ77/p1759831399436109 |
|
@DylanDylann Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
|
Some tests are failing. |
…ert-69844-parasharrajat/member
…ert-69844-parasharrajat/member
|
Fixed |
|
@DylanDylann Changes done. It's ready for your review. |
|
@parasharrajat @carlosmiceli If we invite a user multiple times, Concierge will send multiple messages. Should we hide the previous Concierge messages when the user clicks “Invite” again, or should we hide all Concierge messages once the user interacts with any of them? Screen.Recording.2025-10-23.at.19.52.35.mov |
|
I believe this is an existing behaviour of the app. If you tag a user on chat manually, you would get the same behaviour. So this not in the scope of this PR. |
|
@DylanDylann Bump for review. |
|
LGTM |
|
@parasharrajat Could you please merge the latest main? |
|
@parasharrajat Do you get this error below? Screen.Recording.2025-11-19.at.23.09.00.mov |
|
@parasharrajat It was caused by our change
|
…ert-69844-parasharrajat/member
|
@DylanDylann Should be fixed now. Looks like some new refactors caused this. |
|
@parasharrajat Failed eslint |
|
Ready @DylanDylann |
DylanDylann
left a comment
There was a problem hiding this comment.
@carlosmiceli All yours
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
|
🚀 Deployed to staging by https://github.com/carlosmiceli in version: 9.2.62-0 🚀
|
|
🚀 Deployed to production by https://github.com/marcaaron in version: 9.2.62-5 🚀
|

Explanation of Change
Fixed Issues
$ #69194
PROPOSAL: #69194
Tests
Offline tests
This feature does not work offline.
QA Steps
same as tests.
--->
// TODO: These must be filled out, or the issue title must include "[No QA]."
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
12.09.2025_20.39.32_REC.mp4
iOS: Native
12.09.2025_21.52.04_REC.mp4
iOS: mWeb Safari
12.09.2025_21.44.42_REC.mp4
MacOS: Chrome / Safari
12.09.2025_20.35.31_REC.mp4
MacOS: Desktop
12.09.2025_20.45.36_REC.mp4